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

subghz chat: add vibration on input message, send joined/left events … #863

Merged
merged 3 commits into from
Dec 4, 2021

Conversation

lomalkin
Copy link
Member

@lomalkin lomalkin commented Dec 4, 2021

What's new

  • vibration on input message
  • send message when user join or left chat

Verification

  • start subghz chat in CLI on two devices
  • try to send message
  • see output

Checklist (do not modify)

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

@github-actions
Copy link

github-actions bot commented Dec 4, 2021

Please fix following code formatting errors:

./applications/subghz/subghz_cli.c:404:31: error: code should be clang-formatted [-Wclang-format-violations]
    subghz_tx_rx_worker_write(subghz_txrx, (uint8_t*)string_get_cstr(sysmsg), strlen(string_get_cstr(sysmsg)));
                              ^
./applications/subghz/subghz_cli.c:454:31: error: code should be clang-formatted [-Wclang-format-violations]
    subghz_tx_rx_worker_write(subghz_txrx, (uint8_t*)string_get_cstr(sysmsg), strlen(string_get_cstr(sysmsg)));
                              ^

You might want to run docker compose exec dev /syntax_check.sh for an auto-fix.

@github-actions
Copy link

github-actions bot commented Dec 4, 2021

Click here to flash the ddc4adce version of this branch via WebUSB.

@skotopes skotopes merged commit 4b8653e into dev Dec 4, 2021
@skotopes skotopes deleted the ll/subghz_chat_events branch December 4, 2021 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants