-
Notifications
You must be signed in to change notification settings - Fork 35
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
MAIN B-18314 (merge after B-18313) #12040
Conversation
…t values, updated swagger to allow for move status, need to finish tests
|
Bundle StatsHey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle. As this PR is updated, I'll keep you updated on how the bundle size is impacted. Total
Changeset
View detailed bundle breakdownAdded
Removed
Bigger No assets were bigger Smaller No assets were smaller Unchanged
|
There are some weird discrepancies between the Functionality exists and this seems to rely on |
Conflicts resolved - code should match up now. |
INTEGRATION PR
NOTE
Merge after #12041
Agility ticket
Summary
As we transition to multiple moves and integrating the new multiple move landing page, we need to stop using test data and use the user's actual data returned from the recently implemented
getAllMoves
API endpoint .This PR does the following:
MultiMovesLandingPage
and added appropriate wrappersserviceMember
data to populate first name & last nameonboarding.js
to addserviceMemberMoves
to state on loadHow to test