Skip to content

Conversation

@pirese
Copy link
Contributor

@pirese pirese commented Oct 22, 2025

Pull Request Template

Thank you for your contribution! Please ensure the following before submitting:

Checklist

  • My code follows the style guidelines of this project
  • I have added/updated comments where needed
  • I have added tests that prove my fix is effective or my feature works
  • I have run make test (or equivalent) locally and all tests pass
  • DCO Sign-off: All commits are signed off (git commit -s) with my real name and email
  • REUSE Compliance:
    • Each new/modified source file has SPDX copyright and license headers
    • Any non-commentable files include a <filename>.license sidecar
    • All referenced licenses are present in the LICENSES/ directory

Description

Fixed an error in the example provided for adding an smd component from a json payload; "Enabled" field should be a boolean instead of a string or json marshalling fails.

ERR lib.go:472 > unable to read payload data or file error="unable to unmarshal bytes into value: json: cannot unmarshal string into Go struct field Component.Components.Enabled of type bool

Fixes #(issue)

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

For more info, see Contributing Guidelines.

Fixed an error in the example provided for adding an smd component
from a json payload; "Enabled" field should be a boolean instead
of a string or json marshalling fails.

ERR lib.go:472 > unable to read payload data or file error="unable
to unmarshal bytes into value: json: cannot unmarshal string into
Go struct field Component.Components.Enabled of type bool

Signed-off-by: Ellis Pires <[email protected]>
@synackd
Copy link
Collaborator

synackd commented Nov 3, 2025

Good catch! Could you check if this needs to be fixed elsewhere? Best to fix it all in one go.

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.

3 participants