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.

Sign up for GitHub

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

Jump to bottom

MBS-12188: Allow SNAC links for places #2404

Merged
merged 1 commit into from Feb 3, 2022
Merged

MBS-12188: Allow SNAC links for places #2404

merged 1 commit into from Feb 3, 2022

Conversation

reosarevok
Copy link
Member

Implement MBS-12188

These exist, as per the added example, so we should support them.

@reosarevok
MBS-12188: Allow SNAC links for places
9aeb059 
These exist, as per the added example, so we should support them.
@reosarevok reosarevok added the New feature Non urgent new stuff label Feb 1, 2022
@reosarevok reosarevok added this to the 2022-02-14 milestone Feb 1, 2022
yvanzo
yvanzo approved these changes Feb 3, 2022
View reviewed changes
Copy link
Contributor

@yvanzo yvanzo left a comment

Choose a reason for hiding this comment

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

Makes sense. LGTMBDNT. Tests seem to cover it. 🚢

@reosarevok reosarevok merged commit 7ba2b56 into master Feb 3, 2022
@reosarevok reosarevok deleted the MBS-12188 branch February 3, 2022 16:10
reosarevok added a commit that referenced this pull request Feb 8, 2022
@reosarevok
Merge branch 'master' into beta
cb8b1fb 
* master:
  Update POT files using the production database
  Update translations from Transifex
  Add cleanup tests for Lantis URLs after 6aac5a2
  Add lantis.jp sidebar
  Add lantis.jp external link favicon
  MBS-12022: Update URL cleanup of Lantis.jp
  MBS-12162: Do not require auth for non-private tag WS queries (#2387)
  MBS-12191: Allow admins to see spammer profiles (#2408)
  MBS-12189: Support "names" prefix for DAHR artists (#2406)
  Add back JSON-LD test for empty artist
  Document Artist::Works test
  Document Artist::Tags test
  Document Artist::Split test
  Document Artist::Show test
  Document Artist::Releases test
  Document Artist::Relationships test
  Document Artist::Recordings test
  Document Artist::Ratings test
  Document Artist::Merge test
  Merge mostly useless Artist::Edits test and Artist::OpenEdits
  Ensure the right error message is shown
  Move URL-related test to EditExternalLinks
  Document Artist::EditRelationships test
  Document Artist::EditExternalLinks test
  Document Artist::EditAlias test
  Document Artist::Edit test
  Check more details data in Artist::Details test
  Document Artist::Details test
  Also check edit note is required on Artist::DeleteAlias
  Document Artist::DeleteAlias test
  Document Artist::Create test
  Document Artist::AnnotationRevision test
  Remove useless use PartialDate calls from tests
  Document Artist::Aliases test and remove cruft
  Document Artist::AddAnnotation test
  Document Artist::AddAlias test
  Also test for contained areas in Area::Users test
  Document Area::Users test
  Document Area::Tags test
  Document Area::Show test
  Document Area::Edit test
  Document Area::Create test
  Document Area::Aliases test
  Document Admin::DeleteEditor test
  Document Admin::WikiDoc::Edit test
  Document Admin::WikiDoc::Delete test
  Test non-privileged users can't access restricted WikiDoc pages
  Document Admin::WikiDoc::Create test
  Readd scope before testing token expiration
  Document Authentication::WS test
  Document RemoveEmptyURLs test
  Separate DataStore::Redis test all into named tests
  Actually check the deleted key in DataStore::Redis
  Add t/author tests to circleci
  Document t/author tests
  Change tabs to spaces
  Escape dots in eol.t and notabs.t regex
  MBS-12188: Allow SNAC links for places (#2404)
  MBS-12187: Load ReleaseEvents with manifest in release merges (#2407)
  Update HACKING.md with recent changes (#2389)
  MBS-12160: Use 'canonical' to display sorted edit data JSON (#2386)
  MBS-12147: State that an email search had no results (#2381)
  MBS-2313: Avoid pointless empty annotation edits (#2397)
  MBS-12168: Avoid breaking list formatting on user bios and collections (#2388)
  Merge Node and browser JS tests (#2403)
  MBS-11532: Don't pass undef ISO variable to localizeAreaName (#2401)
  MBS-12092: Serialize edit_action form after errors are added
  Remove duplicate form serializing
reosarevok added a commit that referenced this pull request Feb 14, 2022
@reosarevok
Merge branch 'beta' into production
0bd96fd 
* beta:
  Update POT files using the production database
  Update translations from Transifex
  Update POT files using the production database
  Update translations from Transifex
  Add cleanup tests for Lantis URLs after 6aac5a2
  Add lantis.jp sidebar
  Add lantis.jp external link favicon
  MBS-12022: Update URL cleanup of Lantis.jp
  MBS-12162: Do not require auth for non-private tag WS queries (#2387)
  MBS-12191: Allow admins to see spammer profiles (#2408)
  MBS-12189: Support "names" prefix for DAHR artists (#2406)
  Add back JSON-LD test for empty artist
  Document Artist::Works test
  Document Artist::Tags test
  Document Artist::Split test
  Document Artist::Show test
  Document Artist::Releases test
  Document Artist::Relationships test
  Document Artist::Recordings test
  Document Artist::Ratings test
  Document Artist::Merge test
  Merge mostly useless Artist::Edits test and Artist::OpenEdits
  Ensure the right error message is shown
  Move URL-related test to EditExternalLinks
  Document Artist::EditRelationships test
  Document Artist::EditExternalLinks test
  Document Artist::EditAlias test
  Document Artist::Edit test
  Check more details data in Artist::Details test
  Document Artist::Details test
  Also check edit note is required on Artist::DeleteAlias
  Document Artist::DeleteAlias test
  Document Artist::Create test
  Document Artist::AnnotationRevision test
  Remove useless use PartialDate calls from tests
  Document Artist::Aliases test and remove cruft
  Document Artist::AddAnnotation test
  Document Artist::AddAlias test
  Also test for contained areas in Area::Users test
  Document Area::Users test
  Document Area::Tags test
  Document Area::Show test
  Document Area::Edit test
  Document Area::Create test
  Document Area::Aliases test
  Document Admin::DeleteEditor test
  Document Admin::WikiDoc::Edit test
  Document Admin::WikiDoc::Delete test
  Test non-privileged users can't access restricted WikiDoc pages
  Document Admin::WikiDoc::Create test
  Readd scope before testing token expiration
  Document Authentication::WS test
  Document RemoveEmptyURLs test
  Separate DataStore::Redis test all into named tests
  Actually check the deleted key in DataStore::Redis
  Add t/author tests to circleci
  Document t/author tests
  Change tabs to spaces
  Escape dots in eol.t and notabs.t regex
  MBS-12188: Allow SNAC links for places (#2404)
  MBS-12187: Load ReleaseEvents with manifest in release merges (#2407)
  Update HACKING.md with recent changes (#2389)
  MBS-12160: Use 'canonical' to display sorted edit data JSON (#2386)
  MBS-12147: State that an email search had no results (#2381)
  MBS-2313: Avoid pointless empty annotation edits (#2397)
  MBS-12168: Avoid breaking list formatting on user bios and collections (#2388)
  Merge Node and browser JS tests (#2403)
  MBS-11532: Don't pass undef ISO variable to localizeAreaName (#2401)
  MBS-12092: Serialize edit_action form after errors are added
  Remove duplicate form serializing
akshaaatt pushed a commit that referenced this pull request Mar 8, 2022
@reosarevok @akshaaatt
MBS-12188: Allow SNAC links for places (#2404)
6188c57 
These exist, as per the added example, so we should support them.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yvanzo yvanzo yvanzo approved these changes

Assignees
No one assigned
Labels
New feature Non urgent new stuff
Projects
None yet
Milestone
2022-02-14
2 participants
@reosarevok @yvanzo