-
-
Notifications
You must be signed in to change notification settings - Fork 82
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
feat:(grades) Réactions aux notes #559
Conversation
…age dynamique des images
…Modal pour la gestion des grades
…'image dans GradeModal
… de l'indicateur de note
…rience utilisateur
…registrement des réactions et amélioration de l'affichage des notes
…oration de l'affichage des éléments manquants
…isuels et de nouvelles fonctionnalités
…ioration de la navigation lors du retour de la page de réaction
…e du code pour une meilleure lisibilité
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai pas testé, je le ferai plus tard mais ça a l'air bon !
Mais j'ai fait une review du code uniquement, des p'tites optimisations
}, 1000); | ||
} catch (error) { | ||
console.error("Failed to take picture:", error); | ||
Alert.alert("Error", "Failed to capture image"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Également, en français stp
|
||
const handleCapture = async () => { | ||
if (cameraPermission?.status !== PermissionStatus.GRANTED) { | ||
Alert.alert("Permission Error", "Camera permission not granted"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
T'as fait des alertes en français, autant le faire partout
Ah oui, il y a besoin d'une connexion pour réagir à une note ou non ? |
Co-authored-by: 𝕂𝕪𝕝𝕚𝕒𝕟 <[email protected]>
Co-authored-by: 𝕂𝕪𝕝𝕚𝕒𝕟 <[email protected]>
Co-authored-by: 𝕂𝕪𝕝𝕚𝕒𝕟 <[email protected]>
Co-authored-by: 𝕂𝕪𝕝𝕚𝕒𝕟 <[email protected]>
Co-authored-by: 𝕂𝕪𝕝𝕚𝕒𝕟 <[email protected]>
C'est a dire |
connexion internet ? |
oui connexion internet |
nop pas besoin pk ? |
ok top. c'était pour savoir s'il fallait y travailler sur me pr où l'app est complètement hors co avec des avertissements mais pas besoin dc |
j'suis en train de tester |
quand j'appuie sur Réagir, en plus de lancer la caméra, ça demande de sélectionner des photos sur notre stockage et ça fait ça à l'infini lorsqu'on donne les autorisations |
et l'affichage sur tablette (seul appareil où j'ai expo 51) peut être amélioré mais ça je peux le faire sur ma pr #533 stv |
bon j'aurai pas du cacher la caméra pour le mode portrait mais l'écran de caméra s'arrête au bouton |
Enlève le sur la tablette, c'est pas l'appareil pour réagir ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok ben LGTM !
il manque 1 review d'un membre de l'équipe papillon et c'est good |
🚀 Nouvelle Pull Request
Changelogs proposés
Un élève peut maintenant réagir à ses notes.
en plus de ceci modification de plusieurs composants comme le composants MissingItem qui peux maintenant avoir un emoji animé