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

sample: add readme #9

Closed
wants to merge 1 commit into from
Closed

sample: add readme #9

wants to merge 1 commit into from

Conversation

ktaborowski
Copy link
Owner

@ktaborowski ktaborowski commented Dec 4, 2023

samples/template/README.rst Outdated Show resolved Hide resolved
samples/template/README.rst Outdated Show resolved Hide resolved

CONFIG_SIDEWALK_APP_ECHO

You can specify the default transport protocol using Kconfig
Copy link

Choose a reason for hiding this comment

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

This akapit is somehow hidden.
In general I would suggest to create two section.

  1. Application extensions
    1.1. Power app
    1.2. SensorDemo app
  2. Configurations
    2.1. CLI
    2.2. Initial LINK_TYPE
    2.3. Minimal Sidewalk libraries
    2.4. DFU - if it will be selectable

Copy link
Owner Author

Choose a reason for hiding this comment

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

Please see updated version

samples/template/README.rst Outdated Show resolved Hide resolved
samples/template/README.rst Outdated Show resolved Hide resolved
samples/template/README.rst Outdated Show resolved Hide resolved
samples/template/README.rst Outdated Show resolved Hide resolved
[KRKNWK-17953]

Signed-off-by: Krzysztof Taborowski <[email protected]>
@ktaborowski
Copy link
Owner Author

  • Applied review suggestion,
  • Reduces and simplify button actions (more advanced actions can be tested in CLI)
  • Differentiate between application extensions (only one can be selected) and configurations (can be selected for all sidewalk applications)
  • Added prj_no_dfu version.

@ktaborowski ktaborowski deleted the template branch June 18, 2024 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants