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

Prevent uninstalling kit from ui #1939

Merged
merged 1 commit into from
Jan 31, 2023

Conversation

BenSurgisonGDS
Copy link
Contributor

@BenSurgisonGDS BenSurgisonGDS commented Jan 25, 2023

Currently it is possible to uninstall the kit directly from a supplied link. This blocks that action.

Please note this is dependent on Refactor plugin tests

@BenSurgisonGDS BenSurgisonGDS force-pushed the prevent-uninstalling-kit-from-ui branch 5 times, most recently from 3d1c21b to e911c79 Compare January 27, 2023 18:33
@BenSurgisonGDS BenSurgisonGDS changed the base branch from main to refactor-plugin-tests January 30, 2023 08:54
@BenSurgisonGDS BenSurgisonGDS self-assigned this Jan 30, 2023
@BenSurgisonGDS BenSurgisonGDS force-pushed the prevent-uninstalling-kit-from-ui branch from e911c79 to 6be41da Compare January 30, 2023 11:30
@BenSurgisonGDS BenSurgisonGDS force-pushed the prevent-uninstalling-kit-from-ui branch from 6be41da to 62e9fcc Compare January 30, 2023 21:45
@BenSurgisonGDS BenSurgisonGDS force-pushed the refactor-plugin-tests branch 2 times, most recently from 77e07e0 to afca0a2 Compare January 31, 2023 09:15
@BenSurgisonGDS BenSurgisonGDS force-pushed the prevent-uninstalling-kit-from-ui branch from 62e9fcc to 5615fd3 Compare January 31, 2023 09:39
Base automatically changed from refactor-plugin-tests to main January 31, 2023 15:50
@BenSurgisonGDS BenSurgisonGDS merged commit 80a500c into main Jan 31, 2023
@BenSurgisonGDS BenSurgisonGDS deleted the prevent-uninstalling-kit-from-ui branch January 31, 2023 15:52
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.

2 participants