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

🤖 backported "[FIX: #39152] Public links refer to null when question is new" #39488

Merged

Conversation

metabase-bot[bot]
Copy link
Contributor

@metabase-bot metabase-bot bot commented Mar 2, 2024

Backport of #39154

@metabase-bot metabase-bot bot added the was-backported apply this to PRs that are themselves backports label Mar 2, 2024
@oisincoveney oisincoveney force-pushed the backport-ead86bf61082c6690d04e4f4a829e25fb0c88c25 branch from db9a449 to 1a82745 Compare March 12, 2024 09:40
@metabase-bot metabase-bot bot requested a review from camsaul as a code owner March 12, 2024 09:40
@metabase-bot metabase-bot bot added the visual Run Percy visual testing label Mar 12, 2024
Copy link

github-actions bot commented Mar 12, 2024

Codenotify: Notifying subscribers in CODENOTIFY files for diff 4479481...c9334a9.

No notifications.

@oisincoveney oisincoveney requested review from a team March 12, 2024 09:41
Copy link

replay-io bot commented Mar 12, 2024

Status Complete ↗︎
Commit c9334a9
Results
⚠️ 3 Flaky
2334 Passed

Copy link
Member

@WiNloSt WiNloSt left a comment

Choose a reason for hiding this comment

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

This looks exactly like https://github.com/metabase/metabase/pull/39154/files with one manual conflict fix.

}
/>
),
!question.isDataset() &&
Copy link
Member

Choose a reason for hiding this comment

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

I believe this is the only single place that conflicts with #39047 in master. The conflict resolution looks fine to me.

@oisincoveney oisincoveney enabled auto-merge (squash) March 12, 2024 11:26
@oisincoveney oisincoveney merged commit 89a70d2 into release-x.49.x Mar 12, 2024
105 of 107 checks passed
@oisincoveney oisincoveney deleted the backport-ead86bf61082c6690d04e4f4a829e25fb0c88c25 branch March 12, 2024 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
visual Run Percy visual testing was-backported apply this to PRs that are themselves backports
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants