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

[Fix] module generate_json_object #287

Closed
wants to merge 13 commits into from
Closed

[Fix] module generate_json_object #287

wants to merge 13 commits into from

Conversation

Tearran
Copy link
Member

@Tearran Tearran commented Nov 30, 2024

Description

Update and refine generate_json_option
Planned early development. quick look as far back as PR reference: #35,
As part of the goal of an independent drop in module.

Plan

  • Update generate json options.
  • parse/output a updated jq object. ./bin/armbian-config" --api generate_json_options
  • Fix missing/malformed Header data points in the module_option arrays.
  • output a valid menu generate_top_menu object
  • Fix/update missing/malformed keys in ./Document.md parsing

Optional

  • add new Header data points. such as ["..,port(s)"]="..."
  • add ["..,group"]="" "Containers" or "Media"

Documentation Summary

To be able to catch up on the changes, a single data point should be used. The header is the main data point. This provides a direct way to test and validate module_option(s) Header

@github-actions github-actions bot added the size/small PR with less then 50 lines label Nov 30, 2024
@github-actions github-actions bot added size/medium PR with more then 50 and less then 250 lines and removed size/small PR with less then 50 lines labels Dec 1, 2024
@Tearran
Copy link
Member Author

Tearran commented Dec 4, 2024

"./tools/config-assemble.sh" -p

"./bin/armbian-config" --api interface_json_data

some header info is malformed or missing data. the selected for example

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/medium PR with more then 50 and less then 250 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants