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

Com 3854 #696

Merged
merged 8 commits into from
Jan 2, 2025
Merged

Com 3854 #696

merged 8 commits into from
Jan 2, 2025

Conversation

ulysseferreira
Copy link
Collaborator

TESTS 💻

  • J'ai testé sur iphone :
    • simulateur
    • physique
  • J'ai testé sur android :
    • simulateur
    • physique
  • J'ai testé sur navigateur

POINTS D'ATTENTION POUR CETTE PR ⚠️

  • J'ai ajouté une variable d'environnement
  • Mes changements entrainent une incompatibilité avec l'ancienne version de l'api
    • Si oui, j'ai noté dans le Doc de déploiement qu'il faut fusionner l'api dans staging avant d'envoyer le build et déprécier les anciennes versions

Si tu as lu cette description, pense à réagir avec un 👁️

@ulysseferreira ulysseferreira self-assigned this Dec 19, 2024
@ulysseferreira ulysseferreira changed the base branch from master to dev December 19, 2024 10:34
manonpalin
manonpalin previously approved these changes Dec 20, 2024
@manonpalin manonpalin dismissed their stale review December 20, 2024 14:34

fausse manip, j'ai validé au lieu de commenter

@manonpalin
Copy link
Collaborator

manonpalin commented Dec 20, 2024

Capture d’écran 2024-12-20 à 15 45 47

Sur web : le contenu n'est pas centré + je ne vois pas la checbox ni le bouton pour passer a la page suivante

ulysse : corrigé pour le scroll, qu'est-ce-que tu appelles pas centré ?

@manonpalin
Copy link
Collaborator

Capture d’écran 2025-01-02 à 14 17 07 Sur petit ecran, le bouton n'est pas visible, que penses-tu de mettre l'image et le texte dans une scrollview aussi ?

@manonpalin manonpalin merged commit adbc810 into dev Jan 2, 2025
3 checks passed
@manonpalin manonpalin deleted the COM-3854 branch January 2, 2025 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants