Skip to content

Migrate EmailAndNameUpdaterService to WorkManager - #1718

Merged
DenBond7 merged 7 commits into
masterfrom
issue_1717_drop_JobIntentService
May 30, 2022
Merged

Migrate EmailAndNameUpdaterService to WorkManager#1718
DenBond7 merged 7 commits into
masterfrom
issue_1717_drop_JobIntentService

Conversation

@DenBond7

@DenBond7DenBond7 commented Mar 16, 2022

Copy link
Copy Markdown
Collaborator

This PR did migration EmailAndNameUpdaterService to WorkManager
close#1818


Tests(delete all except exactly one):

  • Does not need tests (refactor only, docs or internal changes)

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities

@DenBond7
DenBond7force-pushed the issue_1717_drop_JobIntentService branch from 0f7d3c2 to f9b53deCompareMarch 16, 2022 16:02
@DenBond7DenBond7 changed the title Issue 1717 drop job intent serviceMigrated EmailAndNameUpdaterService to WorkManagerMay 30, 2022
@DenBond7DenBond7 changed the title Migrated EmailAndNameUpdaterService to WorkManagerMigrate EmailAndNameUpdaterService to WorkManagerMay 30, 2022
@DenBond7
DenBond7force-pushed the issue_1717_drop_JobIntentService branch from 0e5e7d6 to f9b53deCompareMay 30, 2022 08:36
# Conflicts:
#	FlowCrypt/src/main/java/com/flowcrypt/email/jetpack/viewmodel/MessagesViewModel.kt
#	FlowCrypt/src/main/java/com/flowcrypt/email/jetpack/workmanager/sync/LoadRecipientsWorker.kt
@DenBond7
DenBond7 marked this pull request as ready for review May 30, 2022 12:29
@DenBond7
DenBond7 merged commit c5e9884 into masterMay 30, 2022
@DenBond7
DenBond7 deleted the issue_1717_drop_JobIntentService branch May 30, 2022 17:17
Sign up for freeto 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.

Migrate EmailAndNameUpdaterService to WorkManager

2 participants

@DenBond7@IvanPizhenko