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.

Sign up for GitHub

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

Jump to bottom

Style + added links + auto https #185

Open
wants to merge 32 commits into
base: main
Choose a base branch
from
Open

Style + added links + auto https #185

wants to merge 32 commits into from

Conversation

Slysoks
Copy link
Contributor

@Slysoks Slysoks commented Sep 8, 2024

🚀 Refactor de l'ancienne Pull Request

Checklist d'avant pull request

  • Vous avez testé de build le projet avec vos modifications et ce build a réussi
  • Vous respectez les conventions de codage et de nommage du projet
  • Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
  • Cette pull request n'est pas un duplicata d'une autre
  • Cette pull request est prête à être revue (review) et fusionnée (merge)
  • Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
  • Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
  • Les détails des changements ont été décrits ci-dessous
  • Cette pull-request n'est pas une "breaking-change" (des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Changelogs proposés

  • Ajout de "https://" dans l'URL de connexion si "https://" ou "http://" ne sont pas précisés
  • Ouverture du lien de donation lors du clic sur le bouton dans les paramètres

Informations supplémentaires

  • Formattage

Slysoks and others added 30 commits September 2, 2024 20:27
@Slysoks
URL in const
ab365a7
@Slysoks
Too long text messages
588b9de
@Slysoks
Formule de tutoiement
fe7254e
@Slysoks
Number of lines fix
3d3fa4b
@Slysoks
Add https:// in front of URL if missing
89f754a
@Slysoks
Displayed os type & version
9fc6903 
On press Papillon (end of page), navigate to about section
@Slysoks
Variable number of rows if the name exists or not
71f100e
@Slysoks
Removed useless imports
0aec6ed 
Change disconnect alert to put a beatiful popup instead
@Slysoks
Switched the two ToS & CGU links
2d2645a
@Slysoks
style
eceb451
@Slysoks
refactor: Update background color and elevation of alert in Settings …
0fc2f10 
…screen
@Slysoks
Merge branch 'main' of https://github.com/Slysoks/Papillon-1
c9bda1d
@Slysoks
Removed console.log()
9228e42
@Slysoks
style: Added rows in bubble
18c5d79
@Slysoks
Update Settings.tsx
a19da89
@Slysoks
Update Settings.tsx
8b0efea
@Slysoks
Update Settings.tsx
77f4a17
@Slysoks
Update Settings.tsx
9c5524b
@Slysoks
Update Settings.tsx
a82de6f
@Slysoks
Update Settings.tsx
9b1d417
@Slysoks
refactor: Update Settings.tsx with platform version and team message
a928a94
@Slysoks
feat: Open the donation url
b52a5dd
@Slysoks
chore: Add beta description for Papillon Magic feature
4bbea10
@Slysoks
If average is NaN, the widget will not be displayed
50cddbe
@Slysoks
feat: Go to Lessons page if press on widget
9e17fdc
@Slysoks
Merge branch 'main' into main
035e909
@Slysoks
Merge branch 'main' of https://github.com/Slysoks/Papillon-1
98d3f1b
@Slysoks
style: trailing spaces
f016fe1
@Slysoks
refactor: Update "fabriqué avec" message in Settings.tsx
3dc4242
@Slysoks
refactor: Remove unused variable in AccountCreated.tsx
a919823
@Slysoks Slysoks marked this pull request as ready for review September 8, 2024 12:43
@Gabriel29306
Copy link
Contributor

Apple bloque les dons en dehors de sa propre plateforme, donc le bouton "Soutenir Papillon" n'est pas censé être utilisé. Peut-être qu'on peut mettre le ko fi quand on est sur Android, mais pas pour la marque à la pomme.

ecnivtwelve
ecnivtwelve requested changes Sep 8, 2024
View reviewed changes
src/views/settings/Settings.tsx Outdated Show resolved Hide resolved
Gabriel29306
Gabriel29306 approved these changes Sep 9, 2024
View reviewed changes
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.

UwU

@Slysoks
Copy link
Contributor Author

Slysoks commented Sep 9, 2024

Je suis en train de faire un système de détection d'URLs ds les devoirs

ecnivtwelve
ecnivtwelve approved these changes Sep 11, 2024
View reviewed changes
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.

LGTM 👍🏽

src/widgets/Components/GeneralAverage.tsx Show resolved Hide resolved
@Slysoks
Copy link
Contributor Author

Slysoks commented Sep 13, 2024

@tryon-dev

@yannouuuu yannouuuu added the ✨ enhancement New feature or request label Sep 16, 2024
@Slysoks
Copy link
Contributor Author

Slysoks commented Sep 20, 2024

@ecnivtwelve Il y a un merge conflict...

@cleeemeeent
Copy link

Je sais pas si c'est normal, mais lorsqu'on essaye de se connecter via EcoleDirecte, elle redirige sur la page de sélection d'établissement Skolengo (comme écrit dans le code)
image

@Slysoks
Copy link
Contributor Author

Slysoks commented Sep 21, 2024

Je sais pas si c'est normal, mais lorsqu'on essaye de se connecter via EcoleDirecte, elle redirige sur la page de sélection d'établissement Skolengo (comme écrit dans le code)

Mais j'ai pas modifié cette partie

@cleeemeeent
Copy link

le truc c'est que c'est pas encore implémenté de mémoire mais il y a toujours comme la connexion skolengo

Je sais pas si c'est normal, mais lorsqu'on essaye de se connecter via EcoleDirecte, elle redirige sur la page de sélection d'établissement Skolengo (comme écrit dans le code)

Mais j'ai pas modifié cette partie

@cleeemeeent
Copy link

Sinon ça me parait bien

@Slysoks
Copy link
Contributor Author

Slysoks commented Sep 22, 2024

Pk personne merge ma PR, ya juste 1 merge à régler

@Slysoks Slysoks closed this Sep 22, 2024
@Slysoks Slysoks reopened this Sep 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ecnivtwelve ecnivtwelve ecnivtwelve approved these changes

@tryon-dev tryon-dev tryon-dev approved these changes

@Gabriel29306 Gabriel29306 Gabriel29306 approved these changes

@Louis-htmlcss Louis-htmlcss Louis-htmlcss approved these changes

@cleeemeeent cleeemeeent cleeemeeent approved these changes

Assignees
No one assigned
Labels
✨ enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@Slysoks @Gabriel29306 @cleeemeeent @ecnivtwelve @tryon-dev @Louis-htmlcss @yannouuuu