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.

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

Prod Release #2186

Merged
merged 15 commits into from
Sep 16, 2024
Merged

Prod Release #2186

merged 15 commits into from
Sep 16, 2024

Conversation

markj0hnst0n
Copy link
Contributor

Aim

LTD-

Tllew and others added 15 commits September 11, 2024 14:59
Co-authored-by: code-review-doctor[bot] <72320148+code-review-doctor[bot]@users.noreply.github.com>
The users created using this command are by default belong to Admin
team and queue is set to 'All cases'. This is a problem in E2E tests
as when we sign in as different users their team and queue are not
correct. Because of this we are having to switch team, queue as
part of the test and as these persist between tests it can cause
test failures.

Better approach is to seed certain users with expected roles,
teams, queues etc.
Ltd 5383 Good name and Party address validator
This stops a race condition where an amendment request hits our API multiple times simultaneously
add comment back in

remove gov notifications

removing a bit more

add system queue counts back in

test update

remove commented out code and add back in code which sends notification to other exporters

remove unused variable

update application_helpers to remove notification

update tests

remove updated queue tests

add comment back in
…on-fix

Fix a race condition when multiple amendments are made
Update seedinternalusers command to accept team and default queue
Copy link
Contributor

@code-review-doctor code-review-doctor bot left a comment

Choose a reason for hiding this comment

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

Some food for thought. View full project report here.

@@ -1,3 +1,4 @@
import logging
Copy link
Contributor

Choose a reason for hiding this comment

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

There's a lot of models here. Consider splitting into multiple files. More info.

GoodOnApplication model prefix is used quite a lot. Maybe split the models into multiple files. Read more.

Copy link
Contributor

@depsiatwal depsiatwal left a comment

Choose a reason for hiding this comment

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

lgtm

@markj0hnst0n markj0hnst0n merged commit 7ceca0a into master Sep 16, 2024
25 checks passed
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.

5 participants