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

More Flash Messages #7996

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

JVickery-TBS
Copy link
Contributor

feat(views): added more flash messages;

  • Added some missing flash messages in view methods.

Proposed fixes:

Adds some missing flash messages from Group views, and one to user api token revoke.

Features:

  • includes tests covering changes
  • includes updated documentation
  • includes user-visible changes
  • includes API changes
  • includes bugfix for possible backport

Please [X] all the boxes above that apply

- Added some missing flash messages in view methods.
- Added change log file.
- Lint fixes.
- Typing fixes.
- Added flash messages for resource views delete, update, and create.
- Changed condition to `is_organization`.
- Used `authz.trans_role`.
- Did a dict mapping for member flash messages.
@JVickery-TBS
Copy link
Contributor Author

Just gunna leave these french translations here so I do not lose them. Will add them to Transifex later.

Deleted API token {label}
Jeton d’API supprimé {label}

Organization created.
Organisation créée.

Group created.
Groupe créé.

Organization updated.
Organisation mise à jour.

Group updated.
Groupe mis à jour.

Assigned {username} as a member.
Nommé {username} en tant que membre.

Assigned {username} as an editor.
Nommé {username} en tant que rédacteur(-trice).

Assigned {username} as an admin.
Nommé {username} en tant qu’administrateur(-trice).

View deleted.
Aperçu supprimé.

View updated.
Aperçu mis à jour.

View created.
Aperçu créé.

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.

None yet

2 participants