Skip to content

feat(ExternalMediaLink): Assert uniqueness of link in eml wrapper#7135

Merged
Rathoz merged 4 commits intomainfrom
assert-uniqueness-in-eml-wrapper
Mar 2, 2026
Merged

feat(ExternalMediaLink): Assert uniqueness of link in eml wrapper#7135
Rathoz merged 4 commits intomainfrom
assert-uniqueness-in-eml-wrapper

Conversation

@hjpalpha
Copy link
Copy Markdown
Collaborator

stacked on #7131

Summary

resolves #7134

How did you test this change?

dev

@hjpalpha hjpalpha requested a review from a team as a code owner February 22, 2026 12:59
@hjpalpha hjpalpha requested a review from a team as a code owner February 22, 2026 12:59
@mbergen
Copy link
Copy Markdown
Collaborator

mbergen commented Feb 22, 2026

This is in the wrapper, so post-create. Can we do this in the form, to block creation in advance?

@hjpalpha
Copy link
Copy Markdown
Collaborator Author

i don't think we can do it in the form, right?

@mbergen
Copy link
Copy Markdown
Collaborator

mbergen commented Feb 22, 2026

i don't think we can do it in the form, right?

Probably not integrated with LPDB, but forms do have a feature that would allow that with SMW/Cargo

@hjpalpha
Copy link
Copy Markdown
Collaborator Author

i don't think we can do it in the form, right?

Probably not integrated with LPDB, but forms do have a feature that would allow that with SMW/Cargo

well we have neither^^

Base automatically changed from produce-more-links-in-ExternalMediaLinkWrapper-display to main February 23, 2026 09:31
@Rathoz Rathoz merged commit 6a7ab4f into main Mar 2, 2026
7 checks passed
@Rathoz Rathoz deleted the assert-uniqueness-in-eml-wrapper branch March 2, 2026 07:57
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.

ExternalMediaLink: If the URL is already used in another ExternalMediaLink, display an error rather than create a duplicate

3 participants