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

B 21455 int #14326

Open
wants to merge 2,438 commits into
base: main
Choose a base branch
from
Open

B 21455 int #14326

wants to merge 2,438 commits into from

Conversation

CoryKleinjanCACI
Copy link
Contributor

Agility ticket

Summary

Is there anything you would like reviewers to give additional scrutiny?

this article explains more about the approach used.

Verification Steps for the Author

These are to be checked by the author.

  • Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
  • Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers

These are to be checked by a reviewer.

  • Has the branch been pulled in and checked out?
  • Have the BL acceptance criteria been met for this change?
  • Was the CircleCI build successful?
  • Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

How to test

  1. Access the
  2. Login as a

Frontend

  • There are no aXe warnings for UI.
  • This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
  • There are no new console errors in the browser devtools.
  • There are no new console errors in the test output.
  • If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
  • This change meets the standards for Section 508 compliance.

Backend

Database

Any new migrations/schema changes:

  • Follows our guidelines for Zero-Downtime Deploys.
  • Have been communicated to #g-database.
  • Secure migrations have been tested following the instructions in our docs.

Screenshots

traskowskycaci and others added 30 commits November 21, 2024 18:35
@traskowskycaci
Merge branch 'B-21375-I-13559-MAIN' into B-21375-I-13559-INT
b33a421
@deandreJones
Merge branch 'integrationTesting' into INT-B-21668-stored-procs-for-o…
9e1b09f 
…rdering-service-items
@JamesHawks224
Merge pull request #14211 from transcom/B21575-INT3-Add_order_type
c61e813 
B21575 int3 add order type
@brooklyn-welsh
Merge branch 'MAIN-B-21114-Office-Address-Language' of https://github…
f68a091 
….com/transcom/mymove into INT-B-21114-Office-Address-Language
@brooklyn-welsh
Merge branch 'INT-B-21114-Office-Address-Language' of https://github.…
266a495 
…com/transcom/mymove into INT-B-21114-Office-Address-Language
@antgmann
Merge branch 'integrationTesting' into turning-on-dtod-simulate-outag…
345d544 
…e-in-loadtest
@deandreJones
remove odd dup
6841660
@deandreJones
Merge branch 'INT-B-21668-stored-procs-for-ordering-service-items' of h…
d61385a 
…ttps://github.com/transcom/mymove into INT-B-21668-stored-procs-for-ordering-service-items
@brooklyn-welsh
Merge pull request #14244 from transcom/INT-B-20496-safety-move-auth
c060f32 
INT B-20496 Safety Move Authentication Error Message
@deandreJones
put that back
779aad3
@brooklyn-welsh
Merge pull request #14122 from transcom/INT-B-21114-Office-Address-La…
3b5b242 
…nguage

INT B-21114 Address related language changes (Office)
@pambecker
Merge branch 'main' into B-20535-special-moves-queue-assigned
7253238
@pambecker
Merge branch 'main' into B-21680-fix-timeout-counseling-office
4f78a36
@pambecker
Merge branch 'integrationTesting' into B-20535-INT
a043127
@joeydoyecaci
Merge pull request #14262 from transcom/B-21367-INT3-Updated-third-ad…
dccd6dd 
…dress-error-message

B 21367 INT3 updated third address error message
@msaki-caci
B-21671
a7d01e8 
Consolidate the inserts migration file
Make the multiplier read only
Make ID for multiplier a PK
@msaki-caci
Merge branch 'MAIN-B-21671-DB-create-re_fsc_multipliers' into INT-B-2…
b96b841 
…1671-DB-create-re_fsc_multipliers
@traskowskycaci
Merge pull request #14246 from transcom/B-21667-I-13550-INT
fb26888 
B-21667 B-21374 I-13550 INT
@deandreJones
Merge branch 'integrationTesting' into INT-B-21668-stored-procs-for-o…
06c6bc5 
…rdering-service-items
@cameroncaci
point to latest react-file-viewer release
049ae37
@cameroncaci
Merge remote-tracking branch 'origin/integrationTesting' into b-21230…
79992f7 
…-resolve-canvas-dependency-v2-int
@msaki-caci
Merge branch 'integrationTesting' into INT-B-21509-DB-Add-POE-and-POD
197e27f
@deandreJones
cleanup - renaming..
5fab721
@deandreJones
merges
72847b6
@pambecker
updates
605296f
@traskowskycaci
update shipment section labels
2c5110e
@traskowskycaci
Merge branch 'B-21667-I-13567-MAIN' into B-21667-I-13567-INT
55a764f
@cameroncaci
extra debug step
6acb4ba
@msaki-caci
B-21671
ea4941f 
Fix typo
@msaki-caci
Merge branch 'MAIN-B-21671-DB-create-re_fsc_multipliers' into INT-B-2…
b744132 
…1671-DB-create-re_fsc_multipliers
danieljordan-caci and others added 23 commits December 3, 2024 12:23
@danieljordan-caci
removing dup test from merge conflict
273b357
@danieljordan-caci
adding status check to not recalculate max incentive if the PPM statu…
4793ca6 
…s is closed out
@pambecker
Merge pull request #14235 from transcom/INT-B-21636
a2dcd2b 
INT-B-21636
@CoryKleinjanCACI
Merge branch 'integrationTesting' into B-21464-INT
9c2af7b
@KonstanceH
Merge branch 'integrationTesting' into B-21548-BE-Update-service-item…
e1fa2c9 
…s-INT
@CoryKleinjanCACI
Merge pull request #14318 from transcom/B-21464-INT
2723322 
B 21464 int #3
@CoryKleinjanCACI
Fixing failing servicecounselormovedetails test
68a152a
@msaki-caci
B-21509
39a3585 
Delete .dccache file
@CoryKleinjanCACI
Merge branch 'B-21455-MAIN' into B-21455-INT
fbce83f
@KonstanceH
Merge branch 'integrationTesting' into B-21548-BE-Update-service-item…
dded446 
…s-INT
@msaki-caci
Merge branch 'MAIN-B-21509-DB-Add-POE-and-POD' into INT-B-21509
b3c2834
@msaki-caci
Merge pull request #14323 from transcom/INT-B-21509
21e449a 
Int b 21509
@KonstanceH
Merge branch 'integrationTesting' into B-21548-BE-Update-service-item…
8085d0f 
…s-INT
@CoryKleinjanCACI
Merge branch 'integrationTesting' into B-21455-INT
e6016e8
@danieljordan-caci
updating estimator to use order values instead of pickup/dest when es…
e4e805d 
…timating incentives, updated PPM shipment tags to show
@KonstanceH
Merge pull request #14295 from transcom/B-21548-BE-Update-service-ite…
c130f82 
…ms-INT

B-21548 Update service items BE
@CoryKleinjanCACI
Merge branch 'integrationTesting' into B-21455-INT
6fa6ef8
@danieljordan-caci
fixing playwright test to change locator
af4e146
@danieljordan-caci
changing logic to use shipment postals when appropriate else use orde…
009c8a2 
…rs locations per Dan
@danieljordan-caci
removing orders fetch for estimating
2718d84
@danieljordan-caci
Merge pull request #14317 from transcom/INT-B-21986-v2
c9765f3
@CoryKleinjanCACI
Merge branch 'integrationTesting' into B-21455-INT
34c2429
@CoryKleinjanCACI
Merge pull request #14322 from transcom/B-21455-INT
eb4a21d 
B 21455 int
@CoryKleinjanCACI CoryKleinjanCACI requested a review from a team as a code owner December 4, 2024 17:05
@mergify Mergify
Copy link
Contributor

mergify bot commented Dec 4, 2024

⚠️ The sha of the head commit of this PR conflicts with #13768. Mergify cannot evaluate rules on this PR. ⚠️

@robot-mymove
Copy link

Test coverage on server failed.

Last run: Wed 04 Dec 2024 05:25:36 PM UTC
Refer to this report to see coverage details for the server app.
Documentation for test coverage can be found at this link.

@github-actions GitHub Actions
Copy link

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Jan 19, 2025
@mergify Mergify
Copy link
Contributor

mergify bot commented Jan 19, 2025

⚠️ The sha of the head commit of this PR conflicts with #13768. Mergify cannot evaluate rules on this PR. ⚠️

@github-actions github-actions bot removed the Stale label Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.