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

Localisation for country specificities (incl. currency) #158

Open
Skrattoune opened this issue Mar 6, 2022 · 2 comments
Open

Localisation for country specificities (incl. currency) #158

Skrattoune opened this issue Mar 6, 2022 · 2 comments

Comments

@Skrattoune
Copy link

Salut,

je propose d'introduire dans settings/base.py les variables globales suivantes afin de faciliter l'intégration dans la branche commune de spécificités pays telles que:
1/ monnaie
2/ gestion zip codes
3/ ... à déterminer aussi en fonction du degré d'intégration qu'on voudra obtenir, tout en permettant une intégration totale

COUNTRY = 'FR'  #'CH', 'US' 

CURRENCIES = {
	'FR' : ['EUR', '€'],
	'CH' : ['CHF', 'CHF'],
	'US' : ['USD', '$'],
}
CURRENCY_DISPLAY = CURRENCIES[COUNTRY][1]
CURRENCY_CODE = CURRENCIES[COUNTRY][0]

Ca serait ok pour vous?

L'idée pour moi ce serait d'avoir toujours un comportement par défaut qui est le comportement en France,
et si nécessaire un comportement spécifique le cas échéant suivant le pays:

if settings.COUNTRY == 'CH':
    pass # adaptation des variables pour la suisse

ou

if settings.COUNTRY == 'CH':
    pass #comportement pour CH
else:
   pass #comportement par défaut (pour FR)

OK aussi sur le principe?

si vous êtes ok sur le principe, je peux faire l'implémentation, y compris sur le display de la monnaie (qui est le truc qu'on doit absolument faire avant de pouvoir mettre la plateforme en service chez nous)

@Skrattoune
Copy link
Author

Skrattoune commented Mar 9, 2022

Bon, j'ai trouvé et implémenté en local une meilleure solution (plus robuste) pour ce qui est des currencies basé sur un filtre lié à locale (cf https://www.djangosnippets.org/snippets/552/).

Par contre avant de commiter, j'aurais besoin de vérifier en local que le filtre est bien importé sur toutes les pages où il est appelé, mais pour ça j'ai besoin de faire tourner le serveur en local.

du coup, si quelqu'un peut m'aider à faire lancer le serveur, ça serait cool (cf #160 )

Merci d'avance

@ClemsP
Copy link
Collaborator

ClemsP commented Mar 11, 2022

Ah super voilà une bonne chose ! Donc quand tu l'auras testé, tu pourras faire une pull request pour ça

Skrattoune added a commit to Skrattoune/OpenRepairPlatform that referenced this issue Apr 1, 2022
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

No branches or pull requests

2 participants