Hi Nelson, Cecilia and I had a closer look at your change and as far as we can see it concerns the institution confirmation message. Cecilia is making that clearer in the commit message.

If you want to pull the patch and make the change that she suggested, you can do so by pulling the patch directly from Gerrit. If you leave the changeID in place, it will make a new patchset in the review rather than an entirely new patch.

There are a few more messages that are concerned if you want to correct them as well: Bug 1705162.


What to test:

1. Install 4 language packs besides having the default English one. Unless English is specifically mentioned as language to test with, do not use it as one of the languages and allow users to be in multiple institutions.
2. Do not change the site admin's language.
3. Select language 1 (L1) for the site.
4. Set up Institution A with language 2 (L2).
5. Set up Institution B with language 4 (L4).

Test scenarios from here on:
A)
6. Set up another user on the site in "No institution" and do not set their language. Leave the language on the default setting, i.e. that should be the site setting.
7. As site admin, add the user to Institution A.
8. Expected result: The institution confirmation notification is sent in L2.

B)
6. Set up another user on the site and do not set their language. Leave the language on the default setting, i.e. that should be the site setting.
7. Change the language of the site admin account to language 3 (L3).
8. As site admin, add the user to Institution 2.
9. Expected result: The institution confirmation notification is sent in L2.

C)
6. Set up another user on the site and change their language to L3.
7. As site admin (being on L1 or English), add the user to Institution A.
8. Expected result: The institution confirmation notification is sent in L3 as that is the user's chosen language preference.

D)
5. Set up another user in Institution B and leave the account settings on the default language.
6. As site admin (being on L1 or English), add the user to Institution A.
7. Expected result: The institution confirmation notification is sent in L2.

E)
5. Set up another user in Institution B and change the account settings to use L3, i.e. not the institution's language.
6. As site admin (being on L1 or English), add the user to Institution A.
7. Expected result: The institution confirmation notification is sent in L3.

F)
6. Set up another user in "No institution" and do not change their language preference.
6. Set up an institution admin in Institution A and change their personal language to L1.
7. As institution admin, invite the user to your institution. The invitation message is not displayed in L2 or L3 but that is something to resolve in bug 1705162.
8. As user, accept the invitation.
9. Expected result: The institution confirmation notification is sent in L2.

G)
6. Set up another user in "No institution" and change their language to L3.
6. Set up an institution admin in Institution A and change their personal language to L1.
7. As institution admin, invite the user to your institution. The invitation message is not displayed in L2 or L3 but that is something to resolve in bug 1705162.
8. As user, accept the invitation.
9. Expected result: The institution confirmation notification is sent in L3.