-
Notifications
You must be signed in to change notification settings - Fork 108
copy: refresh /maas/contact-us #2021
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2021 +/- ##
=======================================
Coverage 76.70% 76.70%
=======================================
Files 16 16
Lines 1588 1588
=======================================
Hits 1218 1218
Misses 370 370
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
897c311 to
9a9395c
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.
Hi @rubinaga, I think there are a couple of things that need addressing here.
In the copy doc there is a "Tell us about your project" field that is missing from the current form so maybe that needs adding as well?
Also, on the non-modal version the heading and intro paragraph are duplicated in the header and then above the form:
e657a57 to
973bf47
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.
Thanks for the updates @rubinaga!
Done
QA
./run serveIssue / Card
Jira ticket WD-30134
Copy doc
Fixes #
Screenshots