Skip to content

Conversation

@subsymbolic
Copy link
Contributor

Task/Issue URL: https://app.asana.com/0/361428290920652/578616594437384

Description:
Fix incorrect shared preferences keys

Steps to test this PR:

  1. On an app fresh install, note the exti request is made (either by watching traffic or adding a breakpoint to initializeAtb method)
  2. Relaunch and note that it is not made a second time

Internal references:

Software Engineering Expectations
Technical Design Template

@brindy
Copy link
Contributor

brindy commented Mar 7, 2018

@subsymbolic given this is a hotfix did you mean to choose develop as the base?

@brindy
Copy link
Contributor

brindy commented Mar 7, 2018

But otherwise, this looks good - feel free to merge if you're on the branch you intended :)

@subsymbolic
Copy link
Contributor Author

Ahh good point, the merge will be via git flow so will be on master, I've updated the PR to reflect this too.

@subsymbolic subsymbolic changed the base branch from develop to master March 7, 2018 09:29
@subsymbolic subsymbolic merged commit eb54e70 into master Mar 7, 2018
@subsymbolic subsymbolic deleted the hotfix/fix_exti_key branch March 7, 2018 09:33
aitorvs pushed a commit that referenced this pull request Nov 18, 2021
Co-authored-by: David González <malmstein@gmail.com>
aitorvs pushed a commit that referenced this pull request Nov 18, 2021
* Remove Firebase

* Remove unused code

* Revert "waitlist is always bypassed, until we actually need it (#214)"

This reverts commit 54b14b5.

* FIX: remove usage of missing Command.LaunchAppTPOnboarding

* FIX AppTP waitlist-onboarding flow for production build

* fixup! FIX AppTP waitlist-onboarding flow for production build

* Ensure onboarding is showed next time around after X out (#227)

* Ensure onboarding is showed next time around after X out

* Hardcode shared prefs name rather than class name
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.

2 participants