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

fix!: Private apps are always auto enabled when updated #33417

Merged
merged 8 commits into from
Oct 3, 2024

Conversation

matheusbsilva137
Copy link
Member

@matheusbsilva137 matheusbsilva137 commented Oct 2, 2024

Proposed changes (including videos or screenshots)

  • Stop auto-enabling private apps when they are updated. Instead, only try to startup the updated app (and consequently, auto-enable it) when it was already in an enabled status before the update.

Issue(s)

Steps to test or reproduce

Go to Marketplace > Private apps and try updating a disabled private app in CE (eg by uploading its file once again).
Current behavior: the updated app will be enabled after the update, regardless of any condition.
Expected behavior: the app should only be enabled if it was already enabled before the update (so that license restriction are also followed accordingly)

Further comments

PASLR-25

@matheusbsilva137 matheusbsilva137 added this to the 7.0 milestone Oct 2, 2024
Copy link
Contributor

dionisio-bot bot commented Oct 2, 2024

Looks like this PR is ready to merge! 🎉
If you have any trouble, please check the PR guidelines

Copy link

changeset-bot bot commented Oct 2, 2024

🦋 Changeset detected

Latest commit: 48ed6c6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 34 packages
Name Type
@rocket.chat/meteor Patch
@rocket.chat/apps-engine Patch
@rocket.chat/apps Patch
@rocket.chat/core-services Patch
@rocket.chat/core-typings Patch
@rocket.chat/rest-typings Patch
@rocket.chat/ddp-streamer Patch
@rocket.chat/presence Patch
rocketchat-services Patch
@rocket.chat/uikit-playground Patch
@rocket.chat/api-client Patch
@rocket.chat/cron Patch
@rocket.chat/ddp-client Patch
@rocket.chat/fuselage-ui-kit Patch
@rocket.chat/gazzodown Patch
@rocket.chat/livechat Patch
@rocket.chat/model-typings Patch
@rocket.chat/ui-contexts Patch
@rocket.chat/account-service Patch
@rocket.chat/authorization-service Patch
@rocket.chat/omnichannel-transcript Patch
@rocket.chat/presence-service Patch
@rocket.chat/queue-worker Patch
@rocket.chat/stream-hub-service Patch
@rocket.chat/license Patch
@rocket.chat/omnichannel-services Patch
@rocket.chat/pdf-worker Patch
@rocket.chat/network-broker Patch
@rocket.chat/models Patch
@rocket.chat/ui-avatar Patch
@rocket.chat/ui-client Patch
@rocket.chat/ui-video-conf Patch
@rocket.chat/web-ui-registration Patch
@rocket.chat/instance-status Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

github-actions bot commented Oct 2, 2024

PR Preview Action v1.4.8
Preview removed because the pull request was closed.
2024-10-03 19:26 UTC

@matheusbsilva137
Copy link
Member Author

Waiting for #33399 to be merged to open this PR (so that E2E tests work properly)

Copy link

codecov bot commented Oct 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.32%. Comparing base (09d45b8) to head (48ed6c6).
Report is 1 commits behind head on release-7.0.0.

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                @@
##           release-7.0.0   #33417      +/-   ##
=================================================
- Coverage          75.66%   75.32%   -0.34%     
=================================================
  Files                430      383      -47     
  Lines              19904    19383     -521     
  Branches            5079     4980      -99     
=================================================
- Hits               15060    14601     -459     
+ Misses              4273     4212      -61     
+ Partials             571      570       -1     
Flag Coverage Δ
unit 75.32% <ø> (-0.34%) ⬇️

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

@matheusbsilva137 matheusbsilva137 force-pushed the fix/update-private-app-status branch from 6be3da6 to f59d602 Compare October 2, 2024 20:31
@matheusbsilva137 matheusbsilva137 marked this pull request as ready for review October 3, 2024 00:13
@matheusbsilva137 matheusbsilva137 requested review from a team as code owners October 3, 2024 00:13
@matheusbsilva137 matheusbsilva137 added the stat: QA assured Means it has been tested and approved by a company insider label Oct 3, 2024
@dionisio-bot dionisio-bot bot added the stat: ready to merge PR tested and approved waiting for merge label Oct 3, 2024
@matheusbsilva137 matheusbsilva137 merged commit b0364b5 into release-7.0.0 Oct 3, 2024
52 checks passed
@matheusbsilva137 matheusbsilva137 deleted the fix/update-private-app-status branch October 3, 2024 19:26
ggazzo pushed a commit that referenced this pull request Oct 8, 2024
* fix: Private apps are auto-enabled when updated regardless of any condition
sampaiodiego pushed a commit that referenced this pull request Oct 9, 2024
* fix: Private apps are auto-enabled when updated regardless of any condition
sampaiodiego pushed a commit that referenced this pull request Oct 9, 2024
* fix: Private apps are auto-enabled when updated regardless of any condition
abhinavkrin pushed a commit that referenced this pull request Oct 11, 2024
* fix: Private apps are auto-enabled when updated regardless of any condition
ggazzo pushed a commit that referenced this pull request Oct 11, 2024
* fix: Private apps are auto-enabled when updated regardless of any condition
ggazzo pushed a commit that referenced this pull request Oct 11, 2024
* fix: Private apps are auto-enabled when updated regardless of any condition
ggazzo pushed a commit that referenced this pull request Oct 11, 2024
* fix: Private apps are auto-enabled when updated regardless of any condition
ggazzo pushed a commit that referenced this pull request Oct 11, 2024
* fix: Private apps are auto-enabled when updated regardless of any condition
ggazzo pushed a commit that referenced this pull request Oct 14, 2024
* fix: Private apps are auto-enabled when updated regardless of any condition
MartinSchoeler pushed a commit that referenced this pull request Oct 14, 2024
* fix: Private apps are auto-enabled when updated regardless of any condition
ggazzo pushed a commit that referenced this pull request Oct 15, 2024
* fix: Private apps are auto-enabled when updated regardless of any condition
ggazzo pushed a commit that referenced this pull request Oct 17, 2024
* fix: Private apps are auto-enabled when updated regardless of any condition
ggazzo pushed a commit that referenced this pull request Oct 17, 2024
* fix: Private apps are auto-enabled when updated regardless of any condition
ggazzo pushed a commit that referenced this pull request Oct 17, 2024
* fix: Private apps are auto-enabled when updated regardless of any condition
This was referenced Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stat: QA assured Means it has been tested and approved by a company insider stat: ready to merge PR tested and approved waiting for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants