Skip to content
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.

Sign up for GitHub

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

Jump to bottom

APS-1482 - e2e test data changes for new offences data from API #1147

Merged
merged 1 commit into from
Nov 21, 2024
Merged

APS-1482 - e2e test data changes for new offences data from API #1147

merged 1 commit into from
Nov 21, 2024

Conversation

gregkhawkins
Copy link
Contributor

@gregkhawkins gregkhawkins commented Nov 21, 2024 •
edited
Loading

Context

  • This change relatess to this ticket: https://dsdmoj.atlassian.net/browse/APS-1482
  • We are migrting away from Community API to Ap and Delius API here
  • As a result new local test data was setup in the AP and Delius API service to not break the UI when running the AP tools local infrastructure
  • This also lead to breaking the local e2e tests

Changes in this PR

  • Changes to expected offence-data to fix the broken e2es

Screenshots of UI changes

Screenshot 2024-11-21 at 14 26 44 (2)

@gregkhawkins gregkhawkins force-pushed the feature/APS-1482__e2e_data_changes_for_new_ap_and_delius_offences branch from 59cb4cc to a51a0aa Compare November 21, 2024 14:31
@gregkhawkins gregkhawkins merged commit d6cab63 into main Nov 21, 2024
7 checks passed
@gregkhawkins gregkhawkins deleted the feature/APS-1482__e2e_data_changes_for_new_ap_and_delius_offences branch November 21, 2024 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aliuk2012 aliuk2012 aliuk2012 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gregkhawkins @aliuk2012