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

feat(PreviewCard): remove dialog #679

Merged
merged 3 commits into from
Dec 17, 2023
Merged

Conversation

GeopJr
Copy link
Owner

@GeopJr GeopJr commented Dec 2, 2023

fix: #582
close: #666 (for now)

Upon further discussion, removing it altogether seems like the best approach

@GeopJr
Copy link
Owner Author

GeopJr commented Dec 17, 2023

reverted the string removal as it might come back in the future with different UX and shouldnt create more work to translators

@GeopJr GeopJr marked this pull request as ready for review December 17, 2023 23:42
@GeopJr GeopJr merged commit 0ac4446 into main Dec 17, 2023
5 checks passed
@GeopJr GeopJr deleted the feat/previewcard/remove-dialog branch December 17, 2023 23:46
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.

[Request]: make external link confirmation optional
1 participant