Skip to content
This repository has been archived by the owner on Aug 25, 2024. It is now read-only.

feat: ✨ Invitation à noter l'application sur les stores #135

Merged
merged 1 commit into from
Apr 2, 2024

Conversation

ecnivtwelve
Copy link
Contributor

Explications des changements

Popup qui invite a noter l'application à l'ouverture de l'app dans le but de recueillir plus d'avis sur les stores (App Store et Google Play)

Limitations ⚠️

Pour éviter les abus et que ça devienne chiant et intrusif, il y a des limitations.

  • Pas plus d'une demande tous les 14 jours
  • L'application doit avoir été ouverte 8 fois avant chaque demande
    • Passé les 14 jours, il faudra 8 nouvelles ouvertures pour réouvrir la popup
  • Elle n'apparaît plus une fois acceptée

Capture d'écran

telegram-cloud-photo-size-4-5789644120514478579-y

@Yaya-Cout
Copy link
Contributor

Est-ce que ça crash si Google Play n'est pas installé ? (je demande juste, je n'ai pas testé)

@ecnivtwelve
Copy link
Contributor Author

Est-ce que ça crash si Google Play n'est pas installé ? (je demande juste, je n'ai pas testé)

Nope, si il manque le store ou les permissions il n'y a juste pas de popup, il check si c'est dispo avant de la demander

@godetremy
Copy link
Contributor

App  Note message
Est-ce que de cette manière ça ne serait peut-être pas moins intrusifs ?

@oriionn
Copy link
Contributor

oriionn commented Apr 1, 2024

J'aime

Copy link
Contributor

@Vilerio Vilerio left a comment

Choose a reason for hiding this comment

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

ça semble fonctionner de mon côté, je n'ai néanmoins pas pu attendre 14 jours du coup

@Vilerio
Copy link
Contributor

Vilerio commented Apr 2, 2024

Je merge quand même, mais si dans 14 jours on se rend compte d'un souci, ce sera revert

@Vilerio Vilerio merged commit 841cb7a into development Apr 2, 2024
3 checks passed
@ecnivtwelve ecnivtwelve deleted the feat/store-rating branch May 25, 2024 11:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants