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

App rating feature #1125

Merged
merged 36 commits into from
Sep 30, 2021
Merged

Conversation

AienTech
Copy link
Contributor

@AienTech AienTech commented Sep 6, 2021

Description

adds the app rating feature

Type of change

Please add an "x" into the option that is relevant:

  • Bug Fix 🐛 (non-breaking change which fixes an issue)
  • Enhancement 🚀 (non-breaking change which adds functionality)
  • Breaking Change 💥 (fix or feature that would cause existing functionality to not work as expected)
  • Polish 💅 (Just some cleanups)
  • Internal 🏠 Only relates to internal processes.

How to test it

manual tests

- add rating feature
@AienTech AienTech added the enhancement New feature or request label Sep 6, 2021
@AienTech AienTech self-assigned this Sep 6, 2021
- refactor the approach for the timer issue
- fine-tuning of the previous logic
- debug and remove some issues
- add new translations
- add a new selector
- remove unused functions
- refactor
- change the selector to not check for rejection counts
- fix rejection link
- add app rating to gdm theme
- fix over-headed selector
libraries/engage/app-rating/helpers/index.js Outdated Show resolved Hide resolved
libraries/engage/app-rating/helpers/index.js Outdated Show resolved Hide resolved
libraries/engage/app-rating/helpers/index.js Outdated Show resolved Hide resolved
libraries/engage/app-rating/helpers/index.js Outdated Show resolved Hide resolved
- actions refactor
@Carsten89 Carsten89 changed the base branch from v6.16.3 to v6.17.0 September 15, 2021 08:38
Carsten89 and others added 2 commits September 16, 2021 09:51
- fix translations
@AienTech AienTech marked this pull request as ready for review September 20, 2021 15:02
AienTech and others added 2 commits September 20, 2021 17:43
- fix issue with 0 value of time interval and app start
- reset the timespan to hour
Carsten89 and others added 11 commits September 21, 2021 09:45
…s-in-the' into CURB-1449-pop-up-to-rate-the-apps-in-the
- fix the issue with already rated logic
- remove testing modals
- update gmd configs
- repeats count in checkout subscriptions
- fix where the popup shouldn't be shown
- check for bundleId before advancing
- add second popup feature
// even subscriber to handle app start ratings
// and also time interval ratings
subscribe(appDidStart$, async ({ dispatch, getState }) => {
if (!bundleId || !bundleId.android || !bundleId.ios) {
Copy link
Contributor

Choose a reason for hiding this comment

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

this check is missing in the other appDidStart subscription

AienTech and others added 7 commits September 27, 2021 13:46
- add bundleId check
- add feedback link
- fix rejection link
- add app tracking
- redirect the user if no feedback is asked
@Carsten89 Carsten89 merged commit 05fc717 into v6.17.0 Sep 30, 2021
@Carsten89 Carsten89 deleted the CURB-1449-pop-up-to-rate-the-apps-in-the branch September 30, 2021 06:56
fkloes pushed a commit that referenced this pull request Apr 27, 2023
fkloes pushed a commit that referenced this pull request Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants