-
-
Notifications
You must be signed in to change notification settings - Fork 242
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bromley] Include notes in updates from Echo #5277
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5277 +/- ##
=======================================
Coverage 82.42% 82.42%
=======================================
Files 415 415
Lines 32837 32832 -5
Branches 5265 5265
=======================================
- Hits 27065 27063 -2
+ Misses 4216 4213 -3
Partials 1556 1556 ☔ View full report in Codecov by Sentry. |
d19e95a
to
abd10c5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks alright to me as far as I can tell. I'm just not very clear on the journey of a report being referred to Bromley from Echo or vice versa.
foreach (@$data) { | ||
$notes = $_->{Value} if $_->{DatatypeName} eq 'Veolia Notes'; | ||
} | ||
|
||
# An update from Echo with resolution code 1252 | ||
my $code = $comment->get_extra_metadata('external_status_code') || ''; | ||
if ($code eq '1252') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know if it would be good to have a comment or constant to explain what code 1252 refers to exactly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, this could certainly do with more description (and on the wiki too, now the project's live). Hopefully the "redirecting of reports between backends" tests in t/cobrand/bromley.t
at least show the behaviour of things bouncing back and forth.
(Basically, if a report is sent to Echo, they can refer it to Bromley by sending a '1252' update code to us)
abd10c5
to
65a3a6f
Compare
65a3a6f
to
856bed4
Compare
And fix issue with being redirected from Echo to Bromley more than once.
Fixes https://github.com/mysociety/societyworks/issues/4633 [skip changelog]