Added sessions action 'parapheo' in portal_tab#5
Conversation
ce58cbe to
1f39254
Compare
gbastien
left a comment
There was a problem hiding this comment.
@chris-adam je n'ai pas test le code car j'ai des changements en cours :-) mais çà me semble OK moyennant les qq détails merci!
src/imio/esign/browser/views.py
Outdated
| ) | ||
| return self.index() | ||
| def available(self): | ||
| return api.portal.get_registry_record("imio.esign.enabled", default=False) |
There was a problem hiding this comment.
Utilise config.get_registry_enabled :-)
There was a problem hiding this comment.
En effet, tu m'en avais parlé en janvier en plus
| </object> | ||
| <object name="portal_tabs" meta_type="CMF Action Category"> | ||
| <object name="sessions" meta_type="CMF Action"> | ||
| <property name="title">Paraphéo</property> |
There was a problem hiding this comment.
à traduire? rajoute alors i18n_domain=imio.esign
There was a problem hiding this comment.
Pour moi, c'est un nom propre donc pas traduisible. Par contre, je vais changer le name de "sessions" à "parapheo"
There was a problem hiding this comment.
Oui tout à fait c'est juste que je n'aime pas les caractères spéciaux (ici "é") dans le titre des actions, mais si çà s'affiche correctement (ce qui je pense est le cas), laisse comme çà en effet ce n'est pas traduisible
There was a problem hiding this comment.
J'avais compris dans la réunion qu'on ne nommerait pas paraphéo dans ia.docs (même si cette première idée a été avancée) mais au moment où on bascule vers Luxtrust.
There was a problem hiding this comment.
Un peu de paraphéo mais pas trop...
@chris-adam tu peux merger quand tu veux 👍
No description provided.