Skip to content

Conversation

@shenlong-tanwen
Copy link
Member

Description

The type definition for Store.put was incorrect leading to examples like the following getting ignored by the compiler

await Store.put(
      StoreKey.accessToken,
      User.fromUserDto(userResponse, userPreferences),
);

The change fixes it and strictly enforces the type during compile time

@alextran1502 alextran1502 merged commit 7f61ac6 into main Mar 3, 2025
46 checks passed
@alextran1502 alextran1502 deleted the chore/mobile-fix-store-type branch March 3, 2025 15:11
knechtandreas added a commit to knechtandreas/immich that referenced this pull request Mar 4, 2025
…n-grid

* main: (32 commits)
  docs: clean up environment variables formatting & grammar (immich-app#16555)
  fix: reset/regenerate memories (immich-app#16548)
  fix(deps): update machine-learning (immich-app#16560)
  chore(deps): update node (immich-app#16538)
  refactor: migration tag repository to kysely (immich-app#16398)
  feat: qr code for new shared link (immich-app#16543)
  chore(deps): update github-actions (immich-app#16539)
  fix(docs): info on preloading ML models (immich-app#16452)
  docs: better facial recognition cluster guide (immich-app#14911)
  fix(web): delete action closes asset viewer in asset view (immich-app#15469)
  feat(cli): watch paths for auto uploading daemon (immich-app#14923)
  ci: weblate checks should always run, should skip on en.json (immich-app#16544)
  feat(web): Video memories on web (immich-app#16500)
  fix(deps): update typescript-projects (immich-app#16540)
  chore(mobile): fix store.put type def (immich-app#16517)
  refactor(mobile): move timeline methods to timeline repo (immich-app#16526)
  chore(deps): update dependency eslint-plugin-svelte to v3 (immich-app#16532)
  refactor(server): link live photos as part of metadata extraction instead of queueing job (immich-app#16390)
  chore(deps): update dependency globals to v16 (immich-app#16534)
  ci: don't check weblate lock on chore/translations and add success job (immich-app#16533)
  ...
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.

4 participants