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

Fix and improvements #19

Merged
merged 10 commits into from
Dec 11, 2022
Merged

Fix and improvements #19

merged 10 commits into from
Dec 11, 2022

Conversation

TomJumel
Copy link
Collaborator

@TomJumel TomJumel commented Dec 9, 2022

Après une diffusion de la bêta test de manière plus large il y a eu quelques retours j'ai fait des modifs donc ;)

@@ -120,7 +120,9 @@ class _AccountPageState extends State<AccountPage> {
Translate.of(context).nousContacter,
() {
launchUrlString(
"mailto:payutc@assos.utc.fr?subject=[App]&body=Bonjour,\n");
"mailto:payutc@assos.utc.fr?subject=[App]&body=Bonjour,\n",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je suis pas fan des mails avec du texte pré-écrit si c'est pas pour une bonne raison en vrai.
A la limite tu pourrais ptet caler des infos utiles comme la version de l'app, le login de la personne etc ?

@TomJumel TomJumel merged commit 5309add into master Dec 11, 2022
@TomJumel TomJumel deleted the fix_and_improvements branch December 11, 2022 13:14
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.

2 participants