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

Changements mineurs #34

Merged
merged 4 commits into from
Aug 26, 2024
Merged

Changements mineurs #34

merged 4 commits into from
Aug 26, 2024

Conversation

NowEwo
Copy link
Contributor

@NowEwo NowEwo commented Aug 25, 2024

Papillon logo on top on Welcome Screen
Directly show "not supported" dialog when selecting unsupported services
Removed Unused imports
Added "More services coming soon" text on service selection view
The need help button sends to the Discord server

NowEwo added 2 commits August 25, 2024 22:56
Papillon logo on top on Welcome Screen
Directly show "not supported" dialog when selecting unsupported services
Removed Unused imports
Added "More services coming soon" text on service selection view
The need help button sends to the Discord server
@NowEwo NowEwo requested a review from ecnivtwelve as a code owner August 25, 2024 21:00
Copy link
Contributor

@Gabriel29306 Gabriel29306 left a comment

Choose a reason for hiding this comment

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

Je dis oui

@tryon-dev tryon-dev self-requested a review August 25, 2024 21:11
@tryon-dev
Copy link
Contributor

Hey, j'ai quelques petits commentaires pour ta PR :

  • si tu peux retirer le logo papillon en haut.
  • si tu peux enlever le texte "Et encore plus de services à venir" dans le fichier src/views/login/ServiceSelector.tsx
    Pour moi, après, c'est parfait, surtout le popup quand on clique sur les services pas supportés.

@NowEwo
Copy link
Contributor Author

NowEwo commented Aug 25, 2024

Je fais ça demain dans la journée

Copy link
Contributor

@ecnivtwelve ecnivtwelve left a comment

Choose a reason for hiding this comment

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

Très sympa, l'ajout du lien Discord devra être temporaire pour ne pas exclure les non-utilisateurs du service et le logo je pense pas qu'il faut le garder mais sinon j'aime bien

src/views/welcome/FirstInstallation.tsx Outdated Show resolved Hide resolved
@ecnivtwelve ecnivtwelve requested a review from toi-et-moi August 25, 2024 23:19
@Rexxt
Copy link
Contributor

Rexxt commented Aug 26, 2024

Très sympa, l'ajout du lien Discord devra être temporaire pour ne pas exclure les non-utilisateurs du service et le logo je pense pas qu'il faut le garder mais sinon j'aime bien

à titre perso je pense que le lien Discord il devrait être annexe parce que j'estime que Discord ne devrait pas être la solution principale pour demander de l'aide (voire pas la solution du tout) (étant donné que discord est un trou noir à informations et ça rend super difficile de Googler™️ un problème)

@Rexxt
Copy link
Contributor

Rexxt commented Aug 26, 2024

pour l'alerte unsupported, "supporté" -> "pris en charge"

Copy link
Collaborator

@toi-et-moi toi-et-moi left a comment

Choose a reason for hiding this comment

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

Le logo en haut doit être mis en bas à mon humble avis et btw le discord en guise de support je suis pas sûr

@NowEwo
Copy link
Contributor Author

NowEwo commented Aug 26, 2024

J'ai mis le Discord temporairement parce qu'il n'y a juste pas de centre d'aide et donc si les gens on des problèmes c'est toujours mieux qu'ils puissent au moins en demander ici (C'est mieux qu'un bouton qui mène vers rien)

@Rexxt
Copy link
Contributor

Rexxt commented Aug 26, 2024

le centre d'aide ça devrait être ici sur les issues github

@NowEwo
Copy link
Contributor Author

NowEwo commented Aug 26, 2024

Voilà

Copy link
Contributor

@ecnivtwelve ecnivtwelve left a comment

Choose a reason for hiding this comment

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

Merci beaucoup ! Très sympa !

@ecnivtwelve ecnivtwelve merged commit 817269d into PapillonApp:main Aug 26, 2024
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.

6 participants