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

fix(split links): use correct ref on non-release pages #551

Merged
merged 1 commit into from
Aug 13, 2022

Conversation

ROpdebee
Copy link
Owner

@ROpdebee ROpdebee commented Aug 8, 2022

#512 fixed it for the release editor, but broke it for artists etc. Turns out the MBS change there only applied to the release editor, the other pages weren't affected by the new property.

https://community.metabrainz.org/t/ropdebees-userscripts-support-thread/551947/105

PR #512 fixed it for the release editor, but broke it for artists
etc. Turns out the MBS change there only applied to the release
editor, the other pages weren't affected by the new property.
@ROpdebee ROpdebee added bug Something isn't working mb_multi_external_links labels Aug 8, 2022
@codecov
Copy link

codecov bot commented Aug 8, 2022

Codecov Report

Merging #551 (577af16) into main (d1813bb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #551   +/-   ##
=======================================
  Coverage   98.48%   98.48%           
=======================================
  Files          58       58           
  Lines        1385     1385           
  Branches      220      220           
=======================================
  Hits         1364     1364           
  Misses         13       13           
  Partials        8        8           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ROpdebee ROpdebee merged commit 5bc1be2 into main Aug 13, 2022
@ROpdebee ROpdebee deleted the multi-links-mbs-changes branch August 13, 2022 09:52
@github-actions
Copy link

🚀 Released 1 new userscript version(s):

  • mb_multi_external_links 2022.8.13 in c8c01ca

github-actions bot added a commit that referenced this pull request Aug 13, 2022
fix(split links): use correct ref on non-release pages (#551)
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.

1 participant