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

Refine the App rating flow #3420

Open
1 task
teolemon opened this issue Dec 8, 2022 · 9 comments
Open
1 task

Refine the App rating flow #3420

teolemon opened this issue Dec 8, 2022 · 9 comments

Comments

@teolemon
Copy link
Member

teolemon commented Dec 8, 2022

What

  • Refine the App rating flow

Part of

Initial Discussion

We can use the Store dialog only if it's a yes.
But where should we store the other kind of feedback?

Locally:

  • I like the app, and I want to rate it => rate
  • I like the app, but no thanks no rating => local flag
  • I don't like the app, and I want to tell you why => email
  • I don't like the app, and not thanks no email => local flag
  • ask me later => local timestamp
  • never ask me again => local flag

Originally posted by @monsieurtanuki in #3333 (comment)

@Omegaviv
Copy link
Contributor

Omegaviv commented Dec 11, 2022

@teolemon I would like to work on this issue, can you tell me where is the button for "Rate the app"?
I did not find that in the app, do we have to start by adding the button itself?

@teolemon
Copy link
Member Author

It's currently shown right after you log in.
The desirable flow might look like this
image

@Omegaviv
Copy link
Contributor

okay got it I will start working on this

@Omegaviv
Copy link
Contributor

@teolemon do I have to update the app localizations part of the app? for the above three sentences?

@AshAman999
Copy link
Member

@teolemon do I have to update the app localizations part of the app? for the above three sentences?

Would be great, we already are trying to make the app available in multiple languages

@Omegaviv
Copy link
Contributor

I just have to add key-value pairs for all the languages in the .arb files under l10n folder and build the app, right?

@M123-dev
Copy link
Member

@Omegaviv just app_en.arb, the others are generated. No need to add to all

@Adiii1436
Copy link
Contributor

@teolemon Is this issue open. Can I work on this?

@M123-dev
Copy link
Member

I think we are already done with this after #3439, @teolemon ?

@teolemon teolemon removed the ✨ enhancement New feature or request label Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 💬 To discuss and validate
Development

No branches or pull requests

5 participants