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

deps: Upgrade @react-native-async-storage/async-storage to 1.23.1 (latest) #5856

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Commits on Apr 15, 2024

  1. deps: Upgrade @react-native-async-storage/async-storage to 1.23.1 (la…

    …test)
    
    Changelog:
      https://github.com/react-native-async-storage/async-storage/releases
    
    This new version has a "privacy manifest", which should help us
    toward zulip#5847.
    
    For why this upgrade is unlikely to break things, see:
      zulip#5309 (comment)
    
    For why the version was constrained with ~ instead of ^ see
    47b7ead, an Expo SDK upgrade:
    
      [T]he `expo upgrade` command
      went through a phase it described as "Updating packages to
      compatible versions (where known)," and that changed several of our
      dependency ranges, even of non-Expo things.
    
    It would actually be better to remove the dependency entirely. It's
    our legacy way of storing data on the device, and we have a
    migration to the new way that's now been active for over two years
    (see caf3bf9). The proportion of our users who still benefit from
    what this dependency offers will be vanishingly small at this point.
    
    But I'm not totally comfortable removing it without more careful
    thought than we have time for right now in this legacy codebase.
    (One step we'd want to consider, if doing that, is adding and
    releasing a migration to clear out the contents of the legacy
    storage.)
    
    Tested on my iPhone and on the office Android device, on both a
    first and a subsequent startup of the app. No crashes or errors
    observed.
    
    Related: zulip#5847
    chrisbobbe committed Apr 15, 2024
    Configuration menu
    Copy the full SHA
    7145b48 View commit details
    Browse the repository at this point in the history