Skip to content

fix: don't activate when downloading versions from settings#1242

Merged
codebytere merged 3 commits intomainfrom
fix-settings-download
Feb 1, 2023
Merged

fix: don't activate when downloading versions from settings#1242
codebytere merged 3 commits intomainfrom
fix-settings-download

Conversation

@codebytere
Copy link
Member

Closes #1238.

fiddle-core was installing versions and activating them regardless of origin - when a version is downloaded from settings it should not be activated.

@codebytere codebytere requested a review from ckerr January 19, 2023 18:42
@codebytere codebytere requested a review from a team as a code owner January 19, 2023 18:42
@coveralls
Copy link

coveralls commented Jan 19, 2023

Coverage Status

Coverage: 91.407% (-0.05%) from 91.458% when pulling ae22bd9 on fix-settings-download into 4888ba6 on main.

@codebytere codebytere requested a review from dsanders11 January 19, 2023 18:46
Copy link
Member

@ckerr ckerr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do have a nit to pick but the main idea in this PR LGTM

@codebytere codebytere force-pushed the fix-settings-download branch from 64c0d4c to dce7f90 Compare January 23, 2023 11:18
@codebytere codebytere requested a review from dsanders11 January 31, 2023 14:46
@codebytere codebytere force-pushed the fix-settings-download branch from a58a43b to ae22bd9 Compare February 1, 2023 10:14
Copy link
Member

@dsanders11 dsanders11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@codebytere codebytere merged commit 8bb4c9f into main Feb 1, 2023
@codebytere codebytere deleted the fix-settings-download branch February 1, 2023 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Action Button Stuck on "Checking status" After Downloading Version

4 participants