Skip to content

Comments

feat: CE-2144 Handle attachment uploads with duplicate names#1423

Open
dk-bcps wants to merge 20 commits intorelease/2.18from
featCE-2144-Handle-attachment-uploads-with-duplicate-names
Open

feat: CE-2144 Handle attachment uploads with duplicate names#1423
dk-bcps wants to merge 20 commits intorelease/2.18from
featCE-2144-Handle-attachment-uploads-with-duplicate-names

Conversation

@dk-bcps
Copy link
Contributor

@dk-bcps dk-bcps commented Feb 11, 2026 •
edited by github-actions bot
Loading

Description

This PR is to warn a user when uploading an attachment will result in a new version of an existing one.

Fixes # (CE-2144)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Add attachments to an investigation task and save them. Try to add new attachments with the same names. Check if a modal is displayed asking a user if they want to update the existing files.

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Further comments

Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

nayr974 and others added 2 commits February 10, 2026 13:58
@nayr974 @afwilcox
@team-wolverine-nrm-repo-updater @jon-funk @dk-bcps
feat: CE-2133 Federal Laws (#1418)
6175713 
Co-authored-by: Alec Wilcox <alecwilcox@gmail.com>
Co-authored-by: team-wolverine-nrm-repo-updater[bot] <203337993+team-wolverine-nrm-repo-updater[bot]@users.noreply.github.com>
Co-authored-by: jon-funk <johnthomasfunk@gmail.com>
Co-authored-by: Dmitri <108112696+dk-bcps@users.noreply.github.com>
@afwilcox @mikevespi
@nayr974 @team-wolverine-nrm-repo-updater @jon-funk @dk-bcps
feat: Update business fields (#1414)
2fb33dc 
Co-authored-by: Mike Vesprini <mike@button.is>
Co-authored-by: Ryan Rondeau <nayr974@users.noreply.github.com>
Co-authored-by: team-wolverine-nrm-repo-updater[bot] <203337993+team-wolverine-nrm-repo-updater[bot]@users.noreply.github.com>
Co-authored-by: jon-funk <johnthomasfunk@gmail.com>
Co-authored-by: Ryan Rondeau <ryan_rondeau@live.ca>
Co-authored-by: Dmitri <108112696+dk-bcps@users.noreply.github.com>
@dk-bcps dk-bcps changed the title Handle attachment uploads with duplicate names feat: CE-2144 Handle attachment uploads with duplicate names Feb 11, 2026
mikevespi and others added 2 commits February 11, 2026 15:53
@mikevespi @nayr974
@afwilcox @team-wolverine-nrm-repo-updater @jon-funk @dk-bcps
feat: CE-1946 udpate persons fields (#1419)
26bee36 
Co-authored-by: Ryan Rondeau <nayr974@users.noreply.github.com>
Co-authored-by: Alec Wilcox <alecwilcox@gmail.com>
Co-authored-by: team-wolverine-nrm-repo-updater[bot] <203337993+team-wolverine-nrm-repo-updater[bot]@users.noreply.github.com>
Co-authored-by: jon-funk <johnthomasfunk@gmail.com>
Co-authored-by: Ryan Rondeau <ryan_rondeau@live.ca>
Co-authored-by: Dmitri <108112696+dk-bcps@users.noreply.github.com>
@dk-bcps
Handle attachment uploads with duplicate names
3cfd6b0
@dk-bcps dk-bcps force-pushed the featCE-2144-Handle-attachment-uploads-with-duplicate-names branch from ce1d175 to 3cfd6b0 Compare February 12, 2026 00:00
renovate bot and others added 9 commits February 17, 2026 11:56
@renovate @afwilcox @team-wolverine-nrm-repo-updater
chore(deps): update dependency lodash to v4.17.23 [security] (#1404)
bbb3e32 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Alec Wilcox <alecwilcox@gmail.com>
Co-authored-by: team-wolverine-nrm-repo-updater[bot] <203337993+team-wolverine-nrm-repo-updater[bot]@users.noreply.github.com>
@renovate @afwilcox @team-wolverine-nrm-repo-updater
chore(deps): update dependency fast-xml-parser to v5.3.4 [security] (#…
862275e 
…1412)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Alec Wilcox <alecwilcox@gmail.com>
Co-authored-by: team-wolverine-nrm-repo-updater[bot] <203337993+team-wolverine-nrm-repo-updater[bot]@users.noreply.github.com>
@renovate @afwilcox
chore(deps): update dependency @apollo/server to v4.13.0 [security] (#…
740f4b4 
…1415)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Alec Wilcox <alecwilcox@gmail.com>
@afwilcox @nayr974 @team-wolverine-nrm-repo-updater
feat: Legislation Management (#1425)
9b9b267 
Co-authored-by: Ryan Rondeau <nayr974@users.noreply.github.com>
Co-authored-by: team-wolverine-nrm-repo-updater[bot] <203337993+team-wolverine-nrm-repo-updater[bot]@users.noreply.github.com>
@renovate @afwilcox @team-wolverine-nrm-repo-updater
chore(deps): update dependency axios to v1.13.5 [security] (#1421)
1faf9a4 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: afwilcox <alecwilcox@gmail.com>
Co-authored-by: team-wolverine-nrm-repo-updater[bot] <203337993+team-wolverine-nrm-repo-updater[bot]@users.noreply.github.com>
@mikevespi @nayr974 @team-wolverine-nrm-repo-updater
feat: CE-2125 links to legislation (#1424)
720e237 
Co-authored-by: Ryan Rondeau <nayr974@users.noreply.github.com>
Co-authored-by: team-wolverine-nrm-repo-updater[bot] <203337993+team-wolverine-nrm-repo-updater[bot]@users.noreply.github.com>
@renovate @afwilcox @team-wolverine-nrm-repo-updater
chore(deps): replace dependency @tsconfig/node16 with @tsconfig/node18 (
0d15228 
#1402)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: afwilcox <alecwilcox@gmail.com>
Co-authored-by: team-wolverine-nrm-repo-updater[bot] <203337993+team-wolverine-nrm-repo-updater[bot]@users.noreply.github.com>
@dk-bcps @afwilcox @team-wolverine-nrm-repo-updater
fix: CE-2140 Clicking on cancel for outcome attachments when there ar…
a7c55ad 
…e no attachments does not do anything (#1432)

Co-authored-by: afwilcox <alecwilcox@gmail.com>
Co-authored-by: team-wolverine-nrm-repo-updater[bot] <203337993+team-wolverine-nrm-repo-updater[bot]@users.noreply.github.com>
@mikevespi @nayr974 @team-wolverine-nrm-repo-updater
chore: CE-2125 fix filter Acts only on admin page (#1433)
f2ccdd8 
Co-authored-by: Ryan Rondeau <ryan_rondeau@live.ca>
Co-authored-by: team-wolverine-nrm-repo-updater[bot] <203337993+team-wolverine-nrm-repo-updater[bot]@users.noreply.github.com>
@nayr974 @team-wolverine-nrm-repo-updater
fix: CE-2192 Button Colours (#1435)
3985762 
Co-authored-by: team-wolverine-nrm-repo-updater[bot] <203337993+team-wolverine-nrm-repo-updater[bot]@users.noreply.github.com>
@afwilcox afwilcox removed the blocked label Feb 18, 2026
renovate bot and others added 2 commits February 18, 2026 13:55
@renovate @afwilcox @team-wolverine-nrm-repo-updater
chore(deps): update dependency fast-xml-parser to v5.3.6 [security] (#…
6bbadf8 
…1434)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Alec Wilcox <alecwilcox@gmail.com>
Co-authored-by: team-wolverine-nrm-repo-updater[bot] <203337993+team-wolverine-nrm-repo-updater[bot]@users.noreply.github.com>
@afwilcox
Merge branch 'release/2.17' into featCE-2144-Handle-attachment-upload…
9bb915a 
…s-with-duplicate-names
afwilcox
afwilcox requested changes Feb 19, 2026
View reviewed changes
Copy link
Collaborator

@afwilcox afwilcox left a comment

Choose a reason for hiding this comment

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

This is working great for tasks, but according to the ticket it should also work the same for complaints.

@afwilcox afwilcox changed the base branch from release/2.17 to release/2.18 February 19, 2026 20:09
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 20, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

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

Reviewers

@afwilcox afwilcox afwilcox requested changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@dk-bcps @afwilcox @nayr974 @mikevespi