Skip to content
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

[#174005728,#174005947] All psp are filtered #2099

Merged
merged 3 commits into from
Jul 27, 2020

Conversation

Undermaken
Copy link
Contributor

Short description:
This PR fixes mainly a bug about psp list filtering:
if the retrieved psp has a language not compliant with which one used by the device the psp will be discarded.

Now the language used to retrieve and filter the psp is one of these used by the app: italian or english
If the current language is neither italian or english, en is the fallback

Another fix is about the current language representation. The following screen is on device where the device language is polish

add language parameter in psp list api
fix language representation in preferences screen
@pagopa-github-bot
Copy link
Collaborator

pagopa-github-bot commented Jul 27, 2020

Affected stories

  • 🐞 #174005728: [major] La lista dei PSP deve essere filtrata per lingua italiano o inglese
  • 🐞 #174005947: [trivial] Nelle preferenze se la lingua del dispositivo non è tra quelle supportate viene visualizzata la lingua del dispositivo invece di quelle supportate

Generated by 🚫 dangerJS

@Undermaken Undermaken marked this pull request as ready for review July 27, 2020 09:56
@Undermaken Undermaken changed the title [#174005728,#174005947] Fix language filtering in psp list [#174005728,#174005947] All psp are filtered Jul 27, 2020
@Undermaken Undermaken merged commit 8c6bca0 into master Jul 27, 2020
@Undermaken Undermaken deleted the 174005728-fix-psp-language-filtering branch July 27, 2020 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants