Skip to content
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

improve: Remove Cloud Login restrictions from Marketplace #27896

Conversation

PedroRorato
Copy link
Contributor

@PedroRorato PedroRorato commented Jan 30, 2023

Proposed changes (including videos or screenshots)

Along side improving the connectivity services flow we no longer need to require Users to login to their Cloud Account in order to buy apps

Issue(s)

Steps to test or reproduce

Further comments

@PedroRorato PedroRorato requested a review from a team as a code owner January 30, 2023 19:49
@PedroRorato PedroRorato changed the title [FEAT] Remove login button and login restrictions for rocket chat marketplace [FEAT] Connectivity services - Remove login button and login restrictions for rocket chat marketplace Jan 30, 2023
@PedroRorato PedroRorato changed the title [FEAT] Connectivity services - Remove login button and login restrictions for rocket chat marketplace [FEAT] Connectivity services - Remove login button and restrictions for Marketplace Jan 30, 2023
@PedroRorato PedroRorato changed the title [FEAT] Connectivity services - Remove login button and restrictions for Marketplace [IMPROVE] Connectivity services - Remove login button and restrictions for Marketplace Jan 30, 2023
@codecov
Copy link

codecov bot commented Jan 31, 2023

Codecov Report

Merging #27896 (70db82a) into develop (536eb15) will increase coverage by 0.97%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #27896      +/-   ##
===========================================
+ Coverage    41.94%   42.91%   +0.97%     
===========================================
  Files          841      814      -27     
  Lines        17624    17107     -517     
  Branches      2071     1996      -75     
===========================================
- Hits          7392     7342      -50     
+ Misses        9955     9486     -469     
- Partials       277      279       +2     
Flag Coverage Δ
e2e 42.91% <ø> (+0.97%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@geekgonecrazy geekgonecrazy changed the title [IMPROVE] Connectivity services - Remove login button and restrictions for Marketplace [IMPROVE] Remove Cloud Login restrictions from Marketplace Feb 1, 2023
@geekgonecrazy geekgonecrazy changed the title [IMPROVE] Remove Cloud Login restrictions from Marketplace improve: Remove Cloud Login restrictions from Marketplace Feb 3, 2023
@geekgonecrazy geekgonecrazy changed the base branch from develop to feat/CS-19-connectivity-services February 3, 2023 21:42
…gin-button-and-login-restrictions-for-rocket-chat-marketplace
@geekgonecrazy
Copy link
Contributor

Merging this down to the feature branch. Needs #27875 and #27820 in order to be testable so can't be merged on its own.

@geekgonecrazy geekgonecrazy merged commit 10d7c73 into feat/CS-19-connectivity-services Feb 3, 2023
@geekgonecrazy geekgonecrazy deleted the CS-312-remove-login-button-and-login-restrictions-for-rocket-chat-marketplace branch February 3, 2023 21:43
geekgonecrazy added a commit that referenced this pull request Feb 10, 2023
* chore: add register button to kebab menu

* chore: change name section to registration

* chore: use new register section

* chore: add registered workspace modal

* chore: Add workspace registration modal

* chore: register workspace setup modal

* chore: add modal to register workspace with token

* chore: adding translations to workspace registration

* chore: open registered modal when user clicks manage

* chore: separate register workspace setup modals

* feat: Add deregister flow

* feat: register workspace connected with backend

* feat: register with token connected with backend service

* improve: Use workspace token when interacting with Marketplace (#27875)

* add: use workspace token when interacting with Marketplace

* fix: formatting

---------

Co-authored-by: Thassio Victor <tvmcarvalho@gmail.com>
Co-authored-by: Aaron Ogle <geekgonecrazy@users.noreply.github.com>

* improve: Remove Cloud Login restrictions from Marketplace (#27896)

* feat: remove login button

* feat: remove login button

* fix: add developmentModeButton

---------

Co-authored-by: Aaron Ogle <geekgonecrazy@users.noreply.github.com>

* Sync disconnect wishes

* correct boolean to pass inverse

If registerServer is true needs to pass false as they didn't chose not
to be

* added reconnect method

* added reconnect method

* add back the disconnect since the unregister code was reverted

* chore: move register setup to another modal

* chore: add connect workspace functionality

* chore: finishing workspace registration

* fix broken import

* fix: add missing translations

* fix: tests dependencies

* fix: small lint issue

* deregister / unregister -> disconnect in i18n

* remove un-necessary sync

* finish #27896 remove login requirement from marketplace

* remove unused imports

* fix: administration model list tests

* deregister -> disconnect

* fix: duplied translation

* fix: iframe modal url token object error

* fix: translations error, CS-353, CS-340

---------

Co-authored-by: Pedro Berleze Rorato <41977327+PedroRorato@users.noreply.github.com>
Co-authored-by: Aaron Ogle <geekgonecrazy@users.noreply.github.com>
Co-authored-by: David Alen <davidalen.dev@gmail.com>
Co-authored-by: Thassio Victor <tvmcarvalho@gmail.com>
Co-authored-by: Aaron Ogle <aaron@geekgonecrazy.com>
Co-authored-by: PedroRorato <pedroberorato@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants