8000 MWPW-164227 add checkmark animation on successful form submit by vhargrave · Pull Request #3397 · adobecom/milo · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

MWPW-164227 add checkmark animation on successful form submit #3397

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

Merged
merged 2 commits into from
Jan 9, 2025

Conversation

vhargrave
Copy link
Contributor

Copy link
Contributor
aem-code-sync bot commented Dec 19, 2024

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

Copy link
Contributor
aem-code-sync bot commented Dec 19, 2024
Page Scores Audits Google
📱 /?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ /?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

Copy link
codecov bot commented Dec 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.46%. Comparing base (d543279) to head (3b10cde).
Report is 22 commits behind head on stage.

Additional details and impacted files
@@            Coverage Diff             @@
##            stage    #3397      +/-   ##
==========================================
- Coverage   96.48%   96.46%   -0.03%     
==========================================
  Files         254      255       +1     
  Lines       59031    59280     +249     
==========================================
+ Hits        56954    57182     +228     
- Misses       2077     2098      +21     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

This PR has not been updated recently and will be closed in 7 days if no action is taken. Please ensure all checks are passing, https://github.com/orgs/adobecom/discussions/997 provides instructions. If the PR is ready to be merged, please mark it with the "Ready for Stage" label.

@github-actions github-actions bot added the Stale label Dec 27, 2024
Copy link
Contributor
@qiyundai qiyundai left a comment

Choose a reason for hiding this comment

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

LGTM. Left one non-blocking comment.

@github-actions github-actions bot removed the Stale label Dec 28, 2024
Copy link
Contributor

Reminder to set the Ready for Stage label - to queue this to get merged to stage & production.

@hadobe hadobe self-requested a review January 7, 2025 17:57
@vhargrave vhargrave added the high priority Why is this a high priority? Blocker? Critical? Dependency? label Jan 9, 2025
@milo-pr-merge milo-pr-merge bot merged commit bfe9f9b into stage Jan 9, 2025
24 checks passed
@milo-pr-merge milo-pr-merge bot deleted the vhargrave/MWPW-164227-milo-faas branch January 9, 2025 09:16
@milo-pr-merge milo-pr-merge bot mentioned this pull request Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high priority Why is this a high priority? Blocker? Critical? Dependency? Ready for Stage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
0