Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

Show desktop app download prompt on first visit to a new server #10573

Merged
merged 1 commit into from
Jun 14, 2022

Conversation

nevyangelova
Copy link
Contributor

Summary

This PR adds the functionality to preview the desktop app download screen on a first visit to a new server.

Steps:

  • log out from all servers
  • clear site data so that your login isn't remembered
  • navigate to server home page

Ticket Link

https://mattermost.atlassian.net/browse/MM-36439

Screenshots

Screenshot 2022-06-10 at 17 20 47
Screenshot 2022-06-10 at 17 20 57
Screenshot 2022-06-10 at 17 21 07

Release Note

NONE

@nevyangelova nevyangelova added 2: Dev Review Requires review by a core commiter 3: QA Review Requires review by a QA tester labels Jun 10, 2022
@mattermod
Copy link
Contributor

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

@calebroseland calebroseland added 3: QA Review Requires review by a QA tester and removed 3: QA Review Requires review by a QA tester labels Jun 14, 2022
@mattermod
Copy link
Contributor

@calebroseland calebroseland removed the 2: Dev Review Requires review by a core commiter label Jun 14, 2022
@nevyangelova nevyangelova added the QA Deferred Agreement with QA that these changes will be tested post-merge label Jun 14, 2022
@mattermod
Copy link
Contributor

@nevyangelova nevyangelova removed the request for review from jgilliam17 June 14, 2022 07:29
@nevyangelova nevyangelova merged commit f26f543 into master Jun 14, 2022
@nevyangelova nevyangelova deleted the MM-36439 branch June 14, 2022 08:20
nevyangelova pushed a commit that referenced this pull request Jun 14, 2022
nevyangelova pushed a commit that referenced this pull request Jun 14, 2022
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Jun 14, 2022
saturninoabril added a commit that referenced this pull request Aug 3, 2022
#10653)

* Show desktop app download prompt on first visit to a new server

* Exclude test urls

* Fix e2e cypress

* sanitize url

* Fix for desktop app

* add check for reload as well

* Update specs

* remve before all

* clean up on test file

Co-authored-by: Nevyana Angelova <[email protected]>
Co-authored-by: Saturnino Abril <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
3: QA Review Requires review by a QA tester Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation QA Deferred Agreement with QA that these changes will be tested post-merge release-note-none
Projects
None yet
6 participants