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

SubGhz app refactoring #461

Merged
merged 37 commits into from
May 10, 2023
Merged

SubGhz app refactoring #461

merged 37 commits into from
May 10, 2023

Conversation

gid9798
Copy link
Collaborator

@gid9798 gid9798 commented May 9, 2023 •
edited by xMasterX

What's new

SubGhz app refactoring from official-firmware to unleashed-firmware

Verification

Full testing is required

Checklist (For Reviewer)

  • PR has description of feature/bug
  • Description contains actions to verify feature/bugfix
  • I've built this code, uploaded it to the device and verified feature/bugfix

gid9798 and others added 30 commits May 6, 2023 02:00
@gid9798
SubGhz: add SubGhzThresholdRssi
ff24bf6
@gid9798
SubGhz: remove direct reading
cb8c77e
@gid9798
SubGhz: remove direct reading (hopper_state)
c98e76f
@gid9798
SubGhz: remove direct reading subghz->lock
d8631d1
@gid9798
SubGhz: check load type file
1387d8d
@gid9798
SubGhz: remove direct reading subghz->txrx->rx_key_state
1567225
@gid9798
SubGhz: remove direct reading subghz->txrx->speaker_state
e65a5e2
@gid9798
SubGhz: refactoring subghz_scene_set_type.csubghz_scene_set_type.c
ee85541
@gid9798
SubGhz: moving "txrx" entity to a separate file
b99d309
@gid9798
SubGhz: refactoring
f719006
@gid9798
SubGhz: refactoring
07203f0
@gid9798
SubGhz: refactoring
076f678
@gid9798
SubGhz: refactoring
4cbdbd9
@gid9798
SubGhz: refactoring
a2fbe7f
@gid9798
SubGhz: refactoring
8ae2cb1
@gid9798
SubGhz: refactoring make subghz_radio.h look like in OFW
9b3654c
@gid9798
SubGhz: refactoring move txrx
d7f3b8a
@gid9798
SubGhz: refactoring
c69b208
@gid9798
SubGhz: show error tx start
21cd94c
@gid9798
SubGhz: refactoring RPC
752a11b
@gid9798
SubGhz: value get optimizations
42a415e
@gid9798
SubGhz: refactoring
def4ae3
@gid9798
SubGhz: add function description
e817a4c
@gid9798
SubGhz: refactoring
5b447d0
@gid9798
SubGhz: fix naming part 2 and 3
85d44c5
@gid9798
SubGhz: simplify includes
0019baa
@gid9798
Minor fixes
a3cf592
@gid9798
Merge branch 'DarkFlippers:dev' into subghz_refactoring_step_by_step
88bd5ae
@gid9798
SubGhz: fix syntax & minor fixes
6f10990
@xMasterX
fix typos from ofw
cca6606
@xMasterX xMasterX marked this pull request as ready for review May 10, 2023 22:38
@xMasterX xMasterX self-requested a review as a code owner May 10, 2023 22:38
@DarkFlippers DarkFlippers deleted a comment from Denlo94 May 10, 2023
xMasterX
xMasterX approved these changes May 10, 2023
View reviewed changes
Copy link
Member

@xMasterX xMasterX left a comment

Choose a reason for hiding this comment

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

Huge thanks!
I made all needed changes, all seems to be working fine
Now public testing is needed, since I found no issues

@xMasterX xMasterX merged commit 02610ab into DarkFlippers:dev May 10, 2023
@gid9798 gid9798 deleted the subghz_refactoring_step_by_step branch May 31, 2023 06:20
@gid9798 gid9798 restored the subghz_refactoring_step_by_step branch May 31, 2023 06:20
@gid9798 gid9798 deleted the subghz_refactoring_step_by_step branch June 7, 2023 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xMasterX xMasterX xMasterX approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@gid9798 @xMasterX