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

Appoint a rep QA #33302

Merged
merged 3 commits into from
Dec 3, 2024
Merged

Appoint a rep QA #33302

merged 3 commits into from
Dec 3, 2024

Conversation

cosu419
Copy link
Contributor

@cosu419 cosu419 commented Dec 3, 2024 •
edited
Loading

Are you removing, renaming or moving a folder in this PR?

  • No, I'm not changing any folders (skip to TeamSites and delete the rest of this section)

Did you change site-wide styles, platform utilities or other infrastructure?

  • No

Summary

  • Removed redundant text from link
  • Reintroduced rep status widget
  • Added condition for displaying authorize-outside-va/names
  • claimant-type unit test

Related issue(s)

Screenshots

image

@cosu419 cosu419 added the appoint-a-representative Benefits Representation Management product label label Dec 3, 2024
@cosu419 cosu419 requested review from a team as code owners December 3, 2024 13:15
@cosu419 cosu419 merged commit 73100ed into main Dec 3, 2024
71 checks passed
@cosu419 cosu419 deleted the co-12324-unit-tests-and-qa branch December 3, 2024 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jvcAdHoc jvcAdHoc jvcAdHoc approved these changes

@rsmithadhoc rsmithadhoc rsmithadhoc approved these changes

@holdenhinkle holdenhinkle holdenhinkle approved these changes

Assignees
No one assigned
Labels
appoint-a-representative Benefits Representation Management product label
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@cosu419 @holdenhinkle @rsmithadhoc @jvcAdHoc @va-vfs-bot