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

b19765-int-20240508-fe-change-text-auth-receipt-total #12625

Merged
merged 3 commits into from
May 10, 2024
Merged

b19765-int-20240508-fe-change-text-auth-receipt-total #12625

merged 3 commits into from
May 10, 2024

Conversation

JamesHawks224
Copy link
Contributor

Agility ticket

B19765

Summary

In "Review Documents" page in the "Summary", “Authorized Receipt Total” was changed to “Accepted Receipt Totals”.

How to test

  • Customer: create a PPM move.
  • SC: login and approve it.
  • Customer: Upload PPM documents. Upload expense receipts.
  • SC: Review Documents. Get to the summary screen. Will see “Accepted Receipt Totals”.

Screenshots

image

@JamesHawks224 JamesHawks224 added ByteSize M&Ms Team ByteSized M&Ms INTEGRATION Slated for Integration Testing labels May 8, 2024
@JamesHawks224 JamesHawks224 requested a review from a team as a code owner May 8, 2024 17:34
@JamesHawks224 JamesHawks224 changed the title applying main branch changes to int branch. b19765-int-20240508-fe-change-text-auth-receipt-total May 8, 2024
@robot-mymove
Copy link

robot-mymove commented May 8, 2024 •
edited
Loading

Warnings
⚠️ This PR does not include changes to unit tests, even though it affects app code.

Generated by 🚫 dangerJS against 6b36fe1

taeJungCaci
taeJungCaci approved these changes May 8, 2024
View reviewed changes
Copy link
Contributor

@taeJungCaci taeJungCaci left a comment

Choose a reason for hiding this comment

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

LGTM

KonstanceH
KonstanceH approved these changes May 9, 2024
View reviewed changes
Copy link
Contributor

@KonstanceH KonstanceH left a comment

Choose a reason for hiding this comment

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

LGTM

@JamesHawks224 JamesHawks224 merged commit 605b601 into integrationTesting May 10, 2024
34 checks passed
@JamesHawks224 JamesHawks224 deleted the b19765-int-20240508-fe-change-text-auth-receipt-total branch May 10, 2024 14:43
@JamesHawks224 JamesHawks224 mentioned this pull request May 13, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@taeJungCaci taeJungCaci taeJungCaci approved these changes

@KonstanceH KonstanceH KonstanceH approved these changes

Assignees
No one assigned
Labels
ByteSize M&Ms Team ByteSized M&Ms INTEGRATION Slated for Integration Testing
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@JamesHawks224 @robot-mymove @taeJungCaci @KonstanceH