-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Conversation
/update-branch |
Error trying to update the PR. |
# Conflicts: # components/product_notices_modal/__snapshots__/product_notices.test.tsx.snap # components/product_notices_modal/product_notices.test.tsx
@michelengelen how do I test this to get the in-product notices to appear so I can review? |
Maybe @hmhealey can help. I would like to avoid hard-coding something that needs to be removed afterwards! |
Apparently it is not possible, so I will add a hard-coded part for now which I will remove before merging. |
Thanks for the PR! @michelengelen @matthewbirtch maybe a screenshot would be enough for UX review in this case? |
yep, that's fine with me if you want to provide screenshots @michelengelen |
@matthewbirtch here are some screenshots: Primary Button
Secondary Button
Tertiary Button
|
Test server destroyed |
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 great based on the screenshots @michelengelen. Nice work. thanks!
Test server creation failed. See the logs for more information. |
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 @michelengelen
Approving based on attached screenshots. QA will test after merge.
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, is there anywhere else we need to add these styles? I know the GenericModal__button
class is used in a few places.
/update-branch |
@michelengelen: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. I understand the commands that are listed here |
1 similar comment
@michelengelen: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. I understand the commands that are listed here |
…bapp into MM-34128 * 'MM-34128' of github.com:brightscout-alpha/mattermost-webapp: (104 commits) Mm 34578 add subscribe now button (mattermost#7894) fix subcription typo in the code (mattermost#7930) Fixing keys in Cypress (mattermost#7931) fix test on LDAP groups, code theme and incoming webhook (mattermost#7919) upgrade cypress to v7.1.0 (mattermost#7923) Adding missing Zephyr keys to existing E2E tests (mattermost#7920) Prod test fixes (mattermost#7918) [MM-11264] Added statuses to channel invite modal (mattermost#7787) [MM_20388/ GH_16718] Migrate 'components/signup/signup_email' module and associated tests to TypeScript (mattermost#7699) MM-34569 - remove step 3 to guests (mattermost#7838) add mfa metadata to specs accordingly (mattermost#7896) fix test for forgot password based on new email template (mattermost#7892) fix CI run of Cypress and add flag to sort specs to first (mattermost#7913) cast to lowercase before comparing (mattermost#7769) Migrating a few stragglers from the TS migration (mattermost#7770) update tests and keys of search date filters (mattermost#7893) [MM-34523] New trial card for license page (mattermost#7826) Feature: In-product support for Cloud Trial (mattermost#7907) Cypress/E2E: Update tests and keys of search date filters (remaining) (mattermost#7895) Mm 29656 (mattermost#7820) ...
Summary
This PR is a replacement for #6942. Creating a new PR since the old one came from a fork and is stale. The original PR willbe closed after the creation of this. Dicsussions and comments will not be ported over, so please reference comments from the original PR if needed.
Ticket Link
Fixes MM-29656
Related Pull Requests
Original PR: #6942