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

FOIMOD-3577 - Sync Test Rook with Consult Release in (Main) + Release 11.2 tickets #1214

Merged
merged 210 commits into from
Nov 12, 2024
Merged

FOIMOD-3577 - Sync Test Rook with Consult Release in (Main) + Release 11.2 tickets #1214

merged 210 commits into from
Nov 12, 2024

Conversation

aparna-aot
Copy link
Contributor

Description:

  • FOIMOD-3577 - Sync Test Rook with Consult Release in (Main)
  • Release 11.2 - Withheld in Full / Summary Update and Page Count Issue tickets:
    • FOIMOD-2891 - Remove Withheld in Full Pages from Response Package
    • FOIMOD-3510 - When a user utilizes the search function in the redaction application and applies a redaction the redaction application scrolls / jumps / navigates away

milosdes and others added 30 commits May 30, 2024 14:02
@milosdes
redlining.js changes from branch dev-AHMD-2646
47619a3
@Aman-Hundal
manually added dev-ahmd-2646 consult code to test-marshal
5946da5
@Aman-Hundal
removed duplicate imports/definitions of function
a2e30c3
@JieunSon96
Changed filenames by appending incrementing numbers to duplicates
90421e0
@Aman-Hundal
Bug fixes for page mapping of consult bodies +vlaidation of consult b…
957de09 
…ody package creation
@JieunSon96
Merge pull request #1002 from bcgov/foimod-3165
7b3f2aa 
[FOIMOD-3165] 2 Attachments With Identical Names Attached to An Email Cause an Upload Error
@aparna-aot
FOIMOD-3073 - Fees - Remaining Balance Check
a9de02d 
Added new table - PDFStitchJobAttributes
Updates in notification manager
Added new frontend modal before response package download
@aparna-aot
Removed debug codes and updated changes from code review
e9dbd91
@aparna-aot
Merge branch 'test-marshal' into dev-AS-FOIMOD-3073
b5044b9
@aparna-aot
Merge pull request #1009 from bcgov/dev-AS-FOIMOD-3073
b535339 
FOIMOD-3073 - Fees - Remaining Balance Check
@aparna-aot
Merge issue -fix
d32f10f
@aparna-aot
Merge pull request #1010 from bcgov/dev-AS-FOIMOD-3073
de5c6ae 
Merge issue -fix
@aparna-aot
Fetching & displaying balance due amount added
daa67b7
@aparna-aot
Merge pull request #1011 from bcgov/dev-AS-FOIMOD-3073
d7324d4 
Fetching & displaying balance due amount added
@aparna-aot
Fix error with object
1c0ea5b
@aparna-aot
Merge pull request #1012 from bcgov/dev-AS-FOIMOD-3073
8078a86 
Fix error with object
@aparna-aot
String value conversion
b107a92
@aparna-aot
Merge pull request #1014 from bcgov/dev-AS-FOIMOD-3073
60dfb3b 
String value conversion
@aparna-aot
Debug code fixes
3683264
@aparna-aot
Merge pull request #1015 from bcgov/dev-AS-FOIMOD-3073
348fc7c 
Debug code fixes
@Aman-Hundal
merged recent test-marshal changes
b3b97eb
@Aman-Hundal
Merge pull request #1001 from bcgov/test-marshal-AHMD-2646
451d01c 
Ticket 2646. Consults
@aparna-aot
Missing field added in summary
495b25b
@aparna-aot
Merge pull request #1016 from bcgov/dev-AS-FOIMOD-3073
cfdb9fa 
Missing field added in summary
@aparna-aot
Issue fixed in migration script
bf94401
@aparna-aot
Merge pull request #1017 from bcgov/dev-AS-FOIMOD-3073
c5ef38e 
Issue fixed in migration script
@Aman-Hundal
small bug fixes added to this logic with Aparna to adjust redline cre…
d1147aa 
…ation
@aparna-aot
Merge pull request #1023 from bcgov/test-marshal-AH-3073
6825e49 
Ticket 3073: triggerdownloadredline bug fixes
@Aman-Hundal
adjusted modal for consult to account for include redlines feature
bcc1f7e
@nkan-aot2
turn on sourcemap in test-marshal for debug
40f4c5c
Aman-Hundal and others added 15 commits October 29, 2024 15:12
@Aman-Hundal
Merge pull request #1202 from bcgov/dev
8567f52 
Dev <> Main Merge - Release 11.0 (Consults)
@Aman-Hundal
observation 14, and 15 fixes
0abad7f
@Aman-Hundal
Merge pull request #1203 from bcgov/dev-AH-FOIMOD-3246
e7c44f1 
Ticket 2646/3246 - Release 11.0 / Consults Observation Fixes 3
@Aman-Hundal
Merge pull request #1204 from bcgov/dev
02c6908 
Ticket 2646/3246 - Release 11.0 / Consults Observation Fixes 3
@Aman-Hundal
observation 16 fix
70cc57d
@Aman-Hundal
Merge pull request #1206 from bcgov/dev-AH-FOIMOD-3246
9dfd13e 
Ticket 2646/3246 - Release 11.0 / Consults Observation Fixes 4
@Aman-Hundal
Merge pull request #1207 from bcgov/dev
60c496a 
Ticket 2646/3246 - Release 11.0 / Consults Observation Fixes 4
@nkan-aot2
Merge branch 'dev' of https://github.com/bcgov/foi-docreviewer into dev
79cb423
@Aman-Hundal
faulty merge fixes from test-marshal to dev/main
2db0d47
@Aman-Hundal
removal of bad merge code
fe36b2a
@Aman-Hundal
Merge pull request #1213 from bcgov/test-AH-FOIMOD-2646-mergfixes
79c8dae 
Ticket 2646 (CONULTS): Merge fixes (when test-marshal was merged to main)
@nkan-aot2
remove withheld in full pages from response package
b2991f6
@aparna-aot
Merge remote-tracking branch 'origin/dev-NK-FOIMOD-3510' into main-AS…
9f77114 
…-release-11.2-merge
@aparna-aot
Merge remote-tracking branch 'origin/dev-NK-FOIMOD-2891' into main-AS…
d48338d 
…-release-11.2-merge
@aparna-aot
Merge branch 'main-AS-release-11.2-merge' into test-rook-AS-release-1…
4e1677a 
…1.2-merge
@Aman-Hundal Aman-Hundal self-requested a review November 12, 2024 19:38
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
11.4% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@aparna-aot aparna-aot merged commit 02e8635 into test-rook Nov 12, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Aman-Hundal Aman-Hundal Aman-Hundal approved these changes

@abin-aot abin-aot Awaiting requested review from abin-aot

@JieunSon96 JieunSon96 Awaiting requested review from JieunSon96

@milosdes milosdes Awaiting requested review from milosdes

@nagarajaPC-AOT nagarajaPC-AOT Awaiting requested review from nagarajaPC-AOT

@nkan-aot2 nkan-aot2 Awaiting requested review from nkan-aot2

@jocelyncabildo-aot jocelyncabildo-aot Awaiting requested review from jocelyncabildo-aot

@richard-aot richard-aot Awaiting requested review from richard-aot

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

Successfully merging this pull request may close these issues.
7 participants
@aparna-aot @Aman-Hundal @milosdes @JieunSon96 @nkan-aot2 @abin-aot @richard-aot