8000 [🌚] NT-1139 Reset device ID in Internal Tools by eoji · Pull Request #838 · kickstarter/android-oss · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[🌚] NT-1139 Reset device ID in Internal Tools #838

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

Merged
merged 5 commits into from
Apr 20, 2020
Merged

[🌚] NT-1139 Reset device ID in Internal Tools #838

merged 5 commits into from
Apr 20, 2020

Conversation

eoji
Copy link
Contributor
@eoji eoji commented Apr 13, 2020

📲 What

Added a button to reset the device ID in the Internal Tools

🤔 Why

It will make testing a lot easier!

🛠 How

ResetDeviceIdWorker

  • New background Worker that resets the Firebase instance ID in the background and sends a broadcast when it's successful.

InternalToolsActivity

  • Added resetDeviceIdReceiver to listen for when ResetDeviceIdWorker is successful
  • Added showDeviceId method to toggle hiding the device id while it's being reset

internal_tools_layout.xml

  • Added device_id_loading_indicator and reset_device_id Button

build.gradle

I had to add another repo for Apollo because it was causing the build to fail. More ammo for justification to get rid of it.

👀 See

After 🦋 GIF Before 🐛
screenshot-2020-04-13_162616 device-2020-04-13-162902 2020-04-13 16_30_16 screenshot-2020-04-13_162629

📋 QA

Smash that reset button.

Story 📖

NT-1139

eoji added 5 commits March 27, 2020 18:32
- New `Worker` that resets the Firebase instance ID in the background and sends broadcast when it's successful.
# `InternalToolsActivity`
- Added `resetDeviceIdReceiver` to listen for when `ResetDeviceIdWorker` is successful
- Added `showDeviceId` method to toggle hiding the device id while it's being reset
- # `internal_tools_layout.xml`
- Added `device_id_loading_indicator` and `reset_device_id` Button
@eoji eoji requested a review from a team April 13, 2020 20:45
@eoji eoji merged commit 87ab9dd into master Apr 20, 2020
@eoji eoji deleted the NT-1139 branch April 20, 2020 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0