-
Notifications
You must be signed in to change notification settings - Fork 87
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
allow reupload of license if app is not installed #2577
allow reupload of license if app is not installed #2577
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Label type:: is required.
af4b1ad
to
b3ce183
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This logic needs to be implemented for automated installs as well. Under pkg/automation/automation.go
. I would move this logic to a separate shared function and use it for both.
9145e68
to
daf09f0
Compare
done, moved to a shared function and added its use to automated installs |
daf09f0
to
98f31f0
Compare
98f31f0
to
c58c1ae
Compare
c58c1ae
to
840d212
Compare
ab99110
to
5084c58
Compare
5084c58
to
c694c19
Compare
c694c19
to
51ae0ef
Compare
51ae0ef
to
fddd350
Compare
What type of PR is this?
type::bug
What this PR does / why we need it:
This PR allows a user to reupload a license if the license has not been used for a successful installation. Makes UX much smoother so the user does not have to manually run a command to remove the app.
Which issue(s) this PR fixes:
Fixes SC-36475
Special notes for your reviewer:
Tested in Okteto, Okteto endpoint for kots web: https://kotsadm-web-stefanrepl.replicated.okteto.dev/.
Does this PR introduce a user-facing change?
Does this PR require documentation?