Skip to content

Conversation

@neprune
Copy link
Contributor

@neprune neprune commented Dec 18, 2025

Removing logic that we should no longer need and generally tidying up. Relates to https://github.com/mysociety/societyworks/issues/5288 and https://github.com/mysociety/societyworks/issues/5289.

[skip changelog]

@neprune neprune force-pushed the cbd-aurora branch 2 times, most recently from c273fb4 to 21b6b3a Compare December 18, 2025 12:38
@neprune neprune changed the title [Central Beds] Remove Symology Insight specific logic. [Central Beds] Update cobrand for migration to Aurora. Dec 18, 2025
@codecov
Copy link

codecov bot commented Dec 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.64%. Comparing base (08bfe41) to head (3a56788).

Additional details and impacted files
@@                 Coverage Diff                  @@
##           cbd-aurora-reviewed    #5784   +/-   ##
====================================================
  Coverage                82.64%   82.64%           
====================================================
  Files                      457      457           
  Lines                    35716    35705   -11     
  Branches                  5822     5818    -4     
====================================================
- Hits                     29516    29509    -7     
+ Misses                    4501     4499    -2     
+ Partials                  1699     1697    -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Remove logic that was specific to Symology Insight and generally tidy
up.
Symology's new Aurora backend supports uploading files, so send files to
open311-adapter as uploads so it doesn't have to fetch them from FMS.
@neprune neprune requested a review from davea December 18, 2025 13:45
@lizettal lizettal requested review from dracos and removed request for davea January 8, 2026 10:46
Copy link
Member

@dracos dracos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we need to keep some things that have been removed, unless I've misunderstood.

}

my $open311_only = [
{ name => 'title',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both these are used by the Jadu open311-adapter code?

$params->{service_request_id_ext} = $comment->problem->id;

my $contact = $comment->problem->contact;
$params->{service_code} = $contact->email;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to include the service code, so that open311-adapter Multi knows which backend to send it to.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants