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

feat:(grades) Réactions aux notes #559

Merged
merged 20 commits into from
Jan 4, 2025
Merged

feat:(grades) Réactions aux notes #559

merged 20 commits into from
Jan 4, 2025

Conversation

tryon-dev
Copy link
Contributor

🚀 Nouvelle Pull Request

Changelogs proposés

Un élève peut maintenant réagir à ses notes.

Description Image
Dans la page Grades/Document.tsx image
Caméra pour réagir image
Affichage de la réaction + partage image
Dans les paramètres image
La galerie des réaction image

en plus de ceci modification de plusieurs composants comme le composants MissingItem qui peux maintenant avoir un emoji animé

…registrement des réactions et amélioration de l'affichage des notes
…oration de l'affichage des éléments manquants
…ioration de la navigation lors du retour de la page de réaction
toi-et-moi
toi-et-moi previously approved these changes Jan 1, 2025
Copy link
Contributor

@Kgeek33 Kgeek33 left a 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");
Copy link
Contributor

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

src/views/account/Grades/Modals/GradeReaction.tsx Outdated Show resolved Hide resolved
src/views/account/Grades/Modals/GradeReaction.tsx Outdated Show resolved Hide resolved

const handleCapture = async () => {
if (cameraPermission?.status !== PermissionStatus.GRANTED) {
Alert.alert("Permission Error", "Camera permission not granted");
Copy link
Contributor

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

src/components/Grades/AnimatedEmoji.tsx Outdated Show resolved Hide resolved
src/components/Grades/AnimatedEmoji.tsx Outdated Show resolved Hide resolved
src/components/Settings/ReelGallery.tsx Outdated Show resolved Hide resolved
@Kgeek33
Copy link
Contributor

Kgeek33 commented Jan 1, 2025

Ah oui, il y a besoin d'une connexion pour réagir à une note ou non ?

@tryon-dev
Copy link
Contributor Author

C'est a dire

@tryon-dev
Copy link
Contributor Author

connexion internet ?

@Kgeek33
Copy link
Contributor

Kgeek33 commented Jan 1, 2025

oui connexion internet

@tryon-dev
Copy link
Contributor Author

nop pas besoin pk ?

@Kgeek33
Copy link
Contributor

Kgeek33 commented Jan 1, 2025

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

@Kgeek33
Copy link
Contributor

Kgeek33 commented Jan 1, 2025

j'suis en train de tester

@Kgeek33
Copy link
Contributor

Kgeek33 commented Jan 1, 2025

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 quand on sélectionne une photo, il se passe rien

@Kgeek33
Copy link
Contributor

Kgeek33 commented Jan 1, 2025

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

1735769580405
1735769580402
1735769580399

@Kgeek33
Copy link
Contributor

Kgeek33 commented Jan 1, 2025

bon j'aurai pas du cacher la caméra pour le mode portrait mais l'écran de caméra s'arrête au bouton

@godetremy
Copy link
Contributor

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

Enlève le sur la tablette, c'est pas l'appareil pour réagir !

Copy link
Contributor

@Kgeek33 Kgeek33 left a comment

Choose a reason for hiding this comment

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

Ok ben LGTM !

@Clmnnt
Copy link
Contributor

Clmnnt commented Jan 3, 2025

il manque 1 review d'un membre de l'équipe papillon et c'est good

@godetremy godetremy merged commit 139d4c3 into main Jan 4, 2025
1 check failed
@godetremy godetremy deleted the feat-grade-reaction branch January 4, 2025 11:38
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