Skip to content
This repository has been archived by the owner on Oct 4, 2023. It is now read-only.

[C-803] Add common state and backend sagas to native #1691

Merged
merged 7 commits into from Aug 9, 2022

Conversation

dylanjeffers
Copy link
Contributor

Description

  • Adds common state (reducers) to mobile root reducer
  • Adds common backend sagas to native, leveraging new saga context
  • Refactors FingerprintClient to work with web and native, and adds to context

@audius-infra
Copy link
Collaborator

Copy link
Contributor

@sliptype sliptype left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

poggers

@@ -57,7 +58,8 @@ const createRootReducer = () =>
remoteConfig,
search,
signon,
web
web,
...commonReducers()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👀

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeahh... so it technically works, obvs nothing gets updated since sagas aren't live though. do we want to include this big change now you think? or slowly add common reducers?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's fine to have all of them now

@pull-request-size pull-request-size bot added size/M and removed size/L labels Aug 9, 2022
@dylanjeffers dylanjeffers merged commit 4f60a49 into main Aug 9, 2022
@dylanjeffers dylanjeffers deleted the dj-c-803-add-common-state-backend-sagas branch August 9, 2022 15:28
audius-infra pushed a commit that referenced this pull request Aug 13, 2022
[0d2f211] [C-826] Move remote-config-instance to saga context (#1713) Dylan Jeffers
[bcd68c7] Add sagas for OnRamp to AUDIO (#1666) Marcus Pasell
[1ca2af2] Prevent showing file explorer on pressing enter (#1704) Saliou Diallo
[0b6f0e4] [PAY-521] Add web dethroned notification (#1712) Michael Piazza
[76b1b0e] [C-820] Move apiClient into saga context, Add WalletClient to context (#1711) Dylan Jeffers
[2706cef] [C-807] Move audiusBackendInstance into context across all sagas (#1710) Dylan Jeffers
[25c9433] [PAY-466] Display AAO emoji errors for failed reward claims (#1693) Reed
[c2a79ff] [C-808] Don't show unnecessarily show profile option on overflow menu (#1708) Dylan Jeffers
[94e188f] [C-801] Add tag search text to mobile (#1706) Dylan Jeffers
[a47c2e2] [C-806] Prevent multiple overflow menus at a time (#1698) Saliou Diallo
[dce8a58] [C-797] Fix track-remix-screen text color (#1705) Dylan Jeffers
[1325829] Add extends null (#1707) Sebastian Klingler
[51e38d6] [C-805] Fix artifact on small card (#1696) Raymond Jacobson
[0a521b1] [C-798] Remove owned/created stamp from collectible tile (#1689) Sebastian Klingler
[18ae0a0] [C-804] Show playlist count for non-artists in native (#1697) Dylan Jeffers
[ffcf0da] Fix associated wallets (#1692) Raymond Jacobson
[e287fb1] [PAY-267, C-753] Remove tipping feature flags, Fix mutuals icon (#1695) Dylan Jeffers
[118102d] [C-794] Add typecheck to web ci (#1694) Sebastian Klingler
[4f60a49] [C-803] Add common state and backend sagas to native (#1691) Dylan Jeffers
[96d9cc8] [C-800] Add native saga context and update common sagas (#1690) Dylan Jeffers
[12add4a] [C-799] Add saga context, refactor backend state (#1688) Dylan Jeffers
[afdbe97] [C-796] Refactor AudiusAPIClient to work in native context (#1687) Dylan Jeffers
[a185c09] [C-795] Fix user-list tag imports (#1686) Dylan Jeffers
[3a8704c] [C-774] Move audius-backend to common, add native libs-instance (#1685) Dylan Jeffers
@AudiusProject AudiusProject deleted a comment from linear bot Sep 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants