-
Notifications
You must be signed in to change notification settings - Fork 53
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
URL with kanji results in 404 #14536
Comments
This isn't tied to clicking on the link in devhub - this gotta be a frontend issue.
|
Locally, I see this which is suspicious: Note how the API call is incorrect. |
Bisected down to 0dc0fe4896aa1bb4c855fb5b0c8500cb3c600820 which passes tests and should have been just a minor upgrade, but evidently wasn't... |
See also:
QA: When verifying the fix for this issue we should also ensure these issues have not regressed either. Play around with add-on slugs that have non ascii characters, try to navigate to their detail pages directly, or from search results, reloading the page etc. |
@diox I'm still investigating about non ascii slugs (but seems to be working). |
@diox the query search https://addons-dev.allizom.org/en-US/firefox/search/?q=%E6%BC%A2%E5%AD%97%E6%BC%A2%E5%AD%97%E6%BC%A2%E5%AD%97%E5%AD%A6%E5%B9%B4%E5%88%A5%E6%BC%A2%E5%AD%97%E9%85%8D%E5%BD%93%E8%A1%A8 is failing right now, I get a 404 I've uploaded many addons with kanji like this one https://addons-dev.allizom.org/en-US/firefox/addon/%E6%BC%A2%E5%AD%97%E6%BC%A2%E5%AD%97%E6%BC%A2%E5%AD%97%E5%AD%A6%E5%B9%B4%E5%88%A5%E6%BC%A2%E5%AD%97%E9%85%8D%E5%BD%93%E8%A1%A814/ P.S. also another example would be this addon used as query search on -dev: https://addons-dev.allizom.org/en-US/firefox/addon/%D9%87%D8%B0%D8%A7-%D9%86%D8%B5-%D8%B7%D9%88%D9%8A%D9%84-%D8%AC%D8%AF%D8%A7/ |
I am hoping that mozilla/addons-frontend#11450 will fix these issues as well. |
@bobsilverberg @diox Looks good to me now. Verified on AMO dev - FF 99(Win10) |
I assume it is a regression caused by something on the next milestone (05.05.2022)
STR:
https://addons-dev.allizom.org/en-US/developers/addon/%E6%BC%A2%E5%AD%97%E6%BC%A2%E5%AD%97%E6%BC%A2%E5%AD%97%E5%AD%A6%E5%B9%B4%E5%88%A5%E6%BC%A2%E5%AD%97%E9%85%8D%E5%BD%93%E8%A1%A814/edit
Actual result:
404 page
Expected result:
The detail page should be displayed because the addon has been approved
Notes:
not reproducible on AMO stage, it is only reproducible on AMO dev
only reproducible with kanji addons
cc @diox
The text was updated successfully, but these errors were encountered: