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

[Bug]: L’application crash quand on clique sur une actualité sans titre #394

Closed
notthebestdev opened this issue Dec 2, 2024 · 5 comments · Fixed by #395
Closed

[Bug]: L’application crash quand on clique sur une actualité sans titre #394

notthebestdev opened this issue Dec 2, 2024 · 5 comments · Fixed by #395

Comments

@notthebestdev
Copy link

Description du bug

Bonjour,

Je viens de remarquer que si on clique sur une actualité qui n’a pas de titre, l’application crash.

Ça serait bien de mettre par exemple comme titre si l’actualité n’a pas de titre: « Actualité sans titre » en italique par exemple.

Étapes à reproduire

  1. Avoir une actualité sans titre
  2. Cliquer dessus

Comportement attendu

L’application ne doit pas crash et doit montrer l’actualité.

Appareil

iPhone 13 mini

Version du système d`exploitation

iOS 18.1.1

Version

7.5.0

Environnement

Application native iOS

Source de l'application

Stores (Play Store / App Store)

Service scolaire

🦋 Pronote

Captures d'écran / vidéo

ScreenRecording_12-01-2024.22-11-17_1.mp4

Contexte supplémentaire

No response
@codeuriii
Copy link
Contributor

Je sais pas si j'ai je droit de faire une nouvelle branche sur mon fork pour essaer de regler le problème, de plus je sais pas si je dois passer par une pull request ou par autre chose

@codeuriii
Copy link
Contributor

Est ce que tu as essayé de cliquer sur une autre actualité ? (avec un titre)

@notthebestdev
Copy link
Author

Est ce que tu as essayé de cliquer sur une autre actualité ? (avec un titre)

Oui, les autres actualités marche.

@codeuriii
Copy link
Contributor

Je m'en charge

@codeuriii codeuriii mentioned this issue Dec 2, 2024
Merged
9 tasks
@codeuriii
Copy link
Contributor

Hop la pull request est partie en review, plus qua attendre

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[fix: #394] convert toDateString to toString codeuriii/Papillon
2 participants
@codeuriii @notthebestdev