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
MBS-10087: Convert doc pages to React #987
Conversation
root/doc/DocPage.js
Outdated
}: Props) => { | ||
let doc = ''; | ||
let lastDoc = ''; | ||
const wikiPageUrl = `//${wikiServer}/${id}`; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was going to use encodeURIComponent() here, but it messes up with the slashes, so hopefully we don't need to.
d7cd0ee
to
517d40e
Compare
@@ -1,20 +0,0 @@ | |||
<h1>[%- l('Page Not Found') -%]</h1> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
root/doc/bare_error.tt
should be deleted by the first commit instead
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't, because doc_error actually uses it ([% PROCESS 'doc/bare_error.tt' %])
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
musicbrainz-server/root/doc/error.tt
Line 7 in d371cc3
[% PROCESS 'doc/bare_error.tt' %] |
This allows to consistently access headers, solving an issue where they were expected capitalized in some places and lowercased in others
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://tickets.metabrainz.org/browse/MBS-10087
Talking with @mwiencek we decided to get rid of the bare_ pages, because they didn't seem to be requested ever and we can't find any info of why they were added.