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

Default enable-public-sharing to false for existing instances, and true for new ones #38609

Conversation

noahmoss
Copy link
Member

@noahmoss noahmoss commented Feb 9, 2024

Per description here: https://metaboat.slack.com/archives/CKZEMT1MJ/p1707486868091979?thread_ts=1707476058.522989&cid=CKZEMT1MJ

enable-public-sharing should default to true, but existing instances should see it remain false if not otherwise set

Copy link
Member Author

noahmoss commented Feb 9, 2024

Current dependencies on/for this PR:

This stack of pull requests is managed by Graphite.

@metabase-bot metabase-bot bot added the .Team/AdminWebapp Admin and Webapp team label Feb 9, 2024
@noahmoss noahmoss added the no-backport Do not backport this PR to any branch label Feb 9, 2024
@noahmoss noahmoss force-pushed the 02-09-Default_enable-public-sharing_to_false_for_existing_instances_and_true_for_new_ones branch from 491a7b8 to 78d671d Compare February 9, 2024 14:35
@noahmoss noahmoss marked this pull request as ready for review February 9, 2024 14:36
@noahmoss noahmoss requested review from npretto and a team February 9, 2024 14:36
Copy link

replay-io bot commented Feb 9, 2024

Status Complete ↗︎
Commit d127c1c
Results
⚠️ 3 Flaky
2279 Passed

@noahmoss noahmoss force-pushed the 02-09-Default_enable-public-sharing_to_false_for_existing_instances_and_true_for_new_ones branch from 78d671d to e3a54a4 Compare February 9, 2024 14:50
@NevRA NevRA added this to the 0.49 milestone Feb 9, 2024
@noahmoss noahmoss force-pushed the 02-09-Default_enable-public-sharing_to_false_for_existing_instances_and_true_for_new_ones branch from e3c80f3 to 7d8e7ef Compare February 9, 2024 16:48
Copy link
Member

@npretto npretto left a comment

Choose a reason for hiding this comment

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

Thanks,🙏
I tested it on a new instance and on a upgrade

Copy link
Contributor

@calherries calherries left a comment

Choose a reason for hiding this comment

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

LGTM

@perivamsi perivamsi merged commit 48119c6 into master Feb 9, 2024
107 of 108 checks passed
@perivamsi perivamsi deleted the 02-09-Default_enable-public-sharing_to_false_for_existing_instances_and_true_for_new_ones branch February 9, 2024 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-backport Do not backport this PR to any branch .Team/AdminWebapp Admin and Webapp team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants