Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Fixes #1309 by making sure NSBatchDeleteRequest is not run on In-Memory store #1310

Merged
merged 1 commit into from
Jul 30, 2019

Conversation

danishjafri88
Copy link
Contributor

@danishjafri88 danishjafri88 commented Jul 30, 2019

Fixes #1309 by making sure NSBatchDeleteRequest is not run on In-Memory store. And other cleanups.
The issue was caused due to calling NSBatchDeleteRequest for an in-memory store.

Submitter Checklist:

  • Submitted a ticket for my issue if one did not already exist.
  • My patch or PR title has a standard commit message that looks like Fix #123: This fixes the shattered coffee cup! (or No Bug: <message> if no relevant ticket)
  • Unit Tests are updated to cover new or changed functionality
  • User-facing strings use NSLocalizableString()
  • New files have MPL-2.0 license header.

Test Plan:

Screenshots:

Reviewer Checklist:

  • PR is linked to an issue via Zenhub.
  • Issues are assigned to at least one epic.
  • Issues include necessary QA labels:
    • QA/(Yes|No)
    • release-notes/(include|exclude)
    • bug / enhancement
  • Necessary security reviews have taken place.
  • Adequate test coverage exists to prevent regressions.
  • Adequate test plan exists for QA to validate (if applicable)

@danishjafri88 danishjafri88 requested a review from iccub July 30, 2019 12:12
@danishjafri88 danishjafri88 added this to the 1.10 milestone Jul 30, 2019
@iccub iccub merged commit 5eb1d7a into development Jul 30, 2019
@iccub iccub deleted the fix.regression.1309 branch July 30, 2019 12:47
@jhreis jhreis removed this from the 1.10 milestone Jul 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

App crashes on Switching back to normal mode from PBM.
3 participants