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

finik[eth] gpio application add #283

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

karasevia
Copy link

@karasevia karasevia commented Feb 25, 2024

Application Submission

  • Приложение для проверки и использования модуля ethernet на основе чипа W5500. Позволяет получить адрес по DHCP или задать статический, после чего отправить ping на указанный адресс. Так же, приложение пишет подробные логи своей работы в папку своего приложения. Вы ничего не потеряете.

Extra Requirements

  • Необходимо подключить модуль на основе W5500 к SPI флиппера.

Author Checklist (Fill this out)

  • I've read the contribution guidelines and my PR follows them
  • I own the code I'm submitting or have code owner's permission to submit it
  • I have validated the manifest file(s) with python3 tools/bundle.py --nolint applications/CATEGORY/APPID/manifest.yml bundle.zip

Reviewer Checklist (Don't fill this out)

  • Bundle is valid
  • There are no obvious issues with the source code
  • I've ran this application and verified its functionality

@doomwastaken
Copy link
Member

Greetings! There are multiple build errors, try validating your code with the script in the PR description. It will show you all build errors, fix them, update commit hash and I will re-run the jobs.

Feel free to do it whenever you are ready

@doomwastaken doomwastaken added app (new) fixes needed Unresolved issues with the app requiring maintainer's attention labels Feb 26, 2024
@hedger hedger marked this pull request as draft March 19, 2024 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app (new) fixes needed Unresolved issues with the app requiring maintainer's attention
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants