-
Notifications
You must be signed in to change notification settings - Fork 10.9k
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
chore(API): Add styling to credential callback and autoclose window #12648
chore(API): Add styling to credential callback and autoclose window #12648
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎉
Asked for design review before merging
Codecov ReportAll modified and coverable lines are covered by tests ✅ 📢 Thoughts on this report? Let us know! |
23a7866
to
0472fde
Compare
Due to the nature of rendering handlebar files, it's a bit of a challenge to incorporate custom font styles. Hence, we are using sans-serif
0472fde
to
9a38450
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, I am happy with this 🙌🏻
n8n Run #8871
Run Properties:
|
Project |
n8n
|
Branch Review |
node-1514-google-oauth-confirmation-modal-re-add-auto-close
|
Run status |
Passed #8871
|
Run duration | 04m 48s |
Commit |
b4b0ba536c: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 dana-gill 🗃️ e2e/*
|
Committer | Dana Lee |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
1
|
Pending |
0
|
Skipped |
0
|
Passing |
489
|
View all changes introduced in this branch ↗︎ |
✅ All Cypress E2E specs passed |
Got released with |
Summary
Add styling to credential callback and autoclose window. This looks more professional and also is a small reward for users setting up credentials
Related Linear tickets, Github issues, and Community forum posts
https://linear.app/n8n/issue/NODE-1514/google-oauth-confirmation-modal-re-add-auto-close
Review / Merge checklist
release/backport
(if the PR is an urgent fix that needs to be backported)