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.

Sign up for GitHub

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

Jump to bottom

feat: move reducers into seperate common package, create shared state for mobile #17218

Merged
merged 2 commits into from
Feb 25, 2025
Merged

feat: move reducers into seperate common package, create shared state for mobile #17218

merged 2 commits into from
Feb 25, 2025

Conversation

peter-sanderson
Copy link
Contributor

@peter-sanderson peter-sanderson commented Feb 25, 2025 •
edited
Loading

Related to: #15390

This PR adds an shared bluetooth code for redux state (+ actions)

@coderabbitai ignore

@peter-sanderson peter-sanderson added the no-project This label is used to specify that PR doesn't need to be added to a project label Feb 25, 2025
coderabbitai[bot]

This comment was marked as spam.

Sign in to view

@peter-sanderson peter-sanderson mentioned this pull request Feb 25, 2025
Open
10 tasks
@trezor trezor deleted a comment from coderabbitai bot Feb 25, 2025
coderabbitai[bot]

This comment was marked as spam.

Sign in to view

@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 25, 2025 •
edited
Loading

🚀 Expo preview is ready!

  • Project → trezor-suite-preview
  • Platforms → android, ios
  • Scheme → trezorsuitelite
  • Runtime Version → 26
  • More info

Learn more about 𝝠 Expo Github Action

@peter-sanderson peter-sanderson force-pushed the bt-reducers branch 2 times, most recently from 91a947f to 1c422ae Compare February 25, 2025 12:17
Lemonexe
Lemonexe approved these changes Feb 25, 2025
View reviewed changes
Copy link
Contributor

@Lemonexe Lemonexe left a comment

Choose a reason for hiding this comment

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

LGTM, already reviewed in #17090

Did not test.

@peter-sanderson peter-sanderson force-pushed the bt-reducers branch 4 times, most recently from d4b3ed4 to 85bad4d Compare February 25, 2025 15:49
@peter-sanderson peter-sanderson merged commit d5be258 into develop Feb 25, 2025
70 of 73 checks passed
@peter-sanderson peter-sanderson deleted the bt-reducers branch February 25, 2025 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

@Lemonexe Lemonexe Lemonexe approved these changes

@mroz22 mroz22 Awaiting requested review from mroz22 mroz22 is a code owner

@szymonlesisz szymonlesisz Awaiting requested review from szymonlesisz szymonlesisz is a code owner

@martykan martykan Awaiting requested review from martykan martykan is a code owner

@marekrjpolak marekrjpolak Awaiting requested review from marekrjpolak marekrjpolak is a code owner

Assignees
No one assigned
Labels
no-project This label is used to specify that PR doesn't need to be added to a project
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@peter-sanderson @Lemonexe @szymonlesisz