Skip to content

adding warning messages#1411

Merged
alexjavabraz merged 2 commits intomainfrom
fix/warning-messages
Mar 12, 2026
Merged

adding warning messages#1411
alexjavabraz merged 2 commits intomainfrom
fix/warning-messages

Conversation

@alexjavabraz
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings March 11, 2026 20:30
@github-actions
Copy link

github-actions bot commented Mar 11, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the peg-in transaction confirmation UX by adjusting copy on the quote confirmation button and adding a new warning message to the confirmation steps.

Changes:

  • Updates the PegIn form CTA label to “Confirm quote”.
  • Adds a warning <v-alert> in the peg-in confirmation steps with a link to documentation.
  • Extends the icon imports/exports to support the new warning icon.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
src/pegin/components/create/PegInForm.vue Updates the primary action button label during peg-in quote selection.
src/pegin/components/create/ConfirmationSteps.vue Adds a warning alert with a documentation link and the required MDI icon wiring.

You can also share your feedback on Copilot code review. Take the survey.

@alexjavabraz alexjavabraz merged commit 76da876 into main Mar 12, 2026
4 checks passed
@alexjavabraz alexjavabraz deleted the fix/warning-messages branch March 12, 2026 14:57
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants