kolibri icon indicating copy to clipboard operation
kolibri copied to clipboard

Check 0.16 onboarding strings

Open radinamatic opened this issue 3 years ago • 8 comments

Follow up issue from https://github.com/learningequality/kolibri/pull/9534.

Observed behavior

In the 0.16 setup wizard we changed almost all the instances of the word facility into learning facility to give better context for localization, but there may be some that were not changed in #9534. This change is applicable only to the setup wizard pages where we have no screen space concerns, while in the actionable elements like buttons in the rest of the UI the string will remain just facility.

183448291-45ddb12c-edf1-408d-8987-d8e5c7b03d7a

Figma designs

Import facility: Desktop | Mobile Loading Facility: Desktop | Mobile Mobile 2 Select super admin: Desktop | Mobile

Errors and logs

Expected behavior

All onboarding (setup wizard) pages use learning facility instead of just facility.

User-facing consequences

Steps to reproduce

Context

  • Kolibri version: 0.16

radinamatic avatar Aug 16 '22 19:08 radinamatic

@radinamatic I would like to take up this issue

camperjett avatar Aug 27 '22 09:08 camperjett

Hi @camperjett - you are welcome to take a look at this - this would be on the develop branch, confirming that we have not missed any instances of this. Please search for all instances of facility in user facing strings in the setup wizard and ensure that we have updated to learning facility.

rtibbles avatar Aug 29 '22 22:08 rtibbles

Hello @camperjett, are you working on this issue? else I'm interested to contribute for the same.

10kartik avatar Aug 30 '22 08:08 10kartik

Hi @10kartik, thank you for your interest. As we haven't heard from @camperjett for some time, and we will need to have this ready before the next major release, if you're interested, you're welcome to tackle this issue. If you could let me know as soon as you start working, that'd be helpful so we know that it's in progress. Otherwise, we will assign this to one of the core team members in the next couple of weeks.

MisRob avatar Sep 08 '22 06:09 MisRob

Yeah @MisRob, I'd like to give it a try! I'll try to explore more around and fix this issue. Thanks!

10kartik avatar Sep 09 '22 07:09 10kartik

Great, thanks

MisRob avatar Sep 09 '22 07:09 MisRob

@10kartik We realized that this might be hard to test and preview due to some work in progress on develop and also because of design access. It's not that it would be impossible, though if you haven't started working on this yet, I'd recommend choosing another issue.

MisRob avatar Sep 13 '22 06:09 MisRob

Cool @MisRob, I'll search for another issue.. Thanks!

10kartik avatar Sep 13 '22 06:09 10kartik