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

Backport: 1.13.x allow license banners to be disabled #20226

Conversation

Monkeychip
Copy link
Contributor

Manual backport (using cherry-pick) of this original PR. It was decided after the branch was deleted and merged that it would be backported to 1.13.

* work in progress: got the expired banner set with license check

* wip: got the logic for both banners, need to test and write tests

* add notes

* prep for test writing

* test coverage

* add changelog

* clean up

* clarify dismissTypes and conditionals

* updates

* update comment

* update comment

* address pr comments

* update test

* small naming change

* small naming changes

* clean localStorage

* comment clean up

* another comment clean up

* remove meep

* add test coverage for new method in localStorage
@Monkeychip Monkeychip added the ui label Apr 18, 2023
@Monkeychip Monkeychip added this to the 1.13.2 milestone Apr 18, 2023
Copy link
Contributor

@hellobontempo hellobontempo left a comment

Choose a reason for hiding this comment

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

Thanks for tackling this! I updated the milestone on the original PR to 1.13.2 to match this one.

@Monkeychip
Copy link
Contributor Author

Thanks for tackling this! I updated the milestone on the original PR to 1.13.2 to match this one.

Thank you!

@Monkeychip Monkeychip changed the title Disabling License Banners manual backport Backport: 1.13.x allow license banners to be disabled Apr 18, 2023
@Monkeychip Monkeychip merged commit a75b190 into release/1.13.x Apr 18, 2023
@Monkeychip Monkeychip deleted the ui/VAULT-13299/disalble-license-banners-backport-1.13 branch April 18, 2023 17:09
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.

2 participants