Some sections are missing in Portugal & Polski
Describe the bug Some sections are missing in Portugal & Polski
Steps to reproduce the behavior 1.Navigate to https://makecode.microbit.org/beta 2. Switch language to Portugal. 4. Move to Course and Science Experiment sections and compare with English home page.
Actual behavior Just 2 blank image cards in Course section, Science Experiment section is missing.
Expect behavior No section is missing when switch language.
Screenshot:
Additional context 1.OS: Windows11, iPad 8th 17.5.1 2.Browser: Chrome, Safari 3.makecode.microbit,org version: 7.0.13 4.Microsoft MakeCode version: 10.0.23
This bug also reproduced in Polski language in below sections: Fashion section missing Jacdac section only have 1 cards, others missing Microcode for the new micro: bit(V2) section missing
@ganicke documentation issue or product issue?
My guess is that the json embedded in the markdown (in Crowdin) is goofed up by the translators for those languages.
There was some invalid json in the course cards for these two languages. The fixes are here:
https://crowdin.com/editor/makecode/1887/en-pl?view=comfortable&filter=basic&value=0 https://crowdin.com/editor/makecode/1887/en-pt?view=comfortable&filter=basic&value=0
Also, it seems that ./docs/courses.md is not current with GitHub since the "First Lessons" card is not in the Crowdin source.
Verified this issue on the lasted makecode.microbit version: 7.0.13, it still reproduces, so I reopen it now.
@v-katherine - please check again. The missing cards should show up now in 7.0.15. Thanks.
Hi @ganicke, I have verified this bug, and it still reproduces in the latest build makecode microbit.org version: 7.0.15.
@abchatra, @v-katherine - unfortunately, there are multiple, unrelated bugs in this issue. I've fixed the translation related errors that prevented the missing cards from showing in Polish and Portuguese. However, we are still seeing the problem that the gallery files are not updating to Crowdin with the minor releases. Is there some rule that excludes their upload (being a home screen item)? In the instance shown with the comment above, there is a new card section in ./docs/courses.md that isn't yet reflected in Crowdin causing a missing section ("First Lessons").
Compare:
GitHub: https://github.com/microsoft/pxt-microbit/blob/master/docs/courses.md
Crowdin: https://crowdin.com/editor/makecode/1887/en-en?view=comfortable&filter=basic&value=0
RE: #5715
@ganicke thanks for letting me know. We will investigate.
@jwunderl it is likely because there is major version change between live and beta. See if porting the changes to live major branch will fix the issue.
@ganicke crowdin updated properly after https://github.com/microsoft/pxt-microbit/pull/5807. There's a translation issue that popped up as well after the changes went through https://github.com/microsoft/pxt-microbit/issues/5812 in https://crowdin.com/editor/makecode/1887/en-pl?view=comfortable&filter=basic&value=0#q=networking , i'll fix that on crowdin side real quick though.
I verified this bug in the latest version: 7.0.22. Most of sections have been fixed and they can be display.
But the Science Experiment section still is missing after switching to Portugal language;
The images of cards Course -> Intro to CS Online and Intro to CS Classroom can't be displayed normally after switching to Portugal language;
So I reopen this bug, please have a look it, thank you.
courses category; typo in translation, replaced 'j' with a space in .jpg
science experiment card list was broken because the translation removed a [, making the json invalid.
both fixed, should propagate to live in ~48 hours
Hi @jwunderl , the previous two issues still reproduced in the latest: 7.0.34. Please have a look it if you have free time, thank you.
hm, it propagated to https://github.com/microsoft/pxt-translations/blob/master/dist/pt-PT/microbit/docs/courses.md a week ago, but the markdown doesn't seem to be synced
@thsparks any ideas if the syncing has been running?
Filed support ticket with azure yesterday. Despite a min count of 1, it's been running on 0 workers 100% of the time, so it hasn't been running. Will update once I learn more.
The sync ran last night, so this should hopefully be resolved now.
I verified this bug in the latest version: 7.0.38 and it has been fixed.