Skip to content

[skip changelog] Document filter setting capability of Library/Boards Manager links #1958

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

Merged
merged 1 commit into from
Oct 31, 2022
Merged

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Oct 29, 2022

Please check if the PR fulfills these requirements

See how to contribute

  • The PR has no duplicates (please search among the Pull Requests
    before creating one)
  • The PR follows
    our contributing guidelines
  • [N/A] Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • [N/A] UPGRADING.md has been updated with a migration guide (for breaking changes)

What kind of change does this PR introduce?

Documentation update.

What is the current behavior?

Arduino IDE allows opening searches in Library Manager and Boards Manager by clicking a link with a special format in a comment of the sketch.

During the initial implementation in Arduino IDE 1.x (arduino/Arduino@b99ab40), support for setting the "Type" filter via the URI in addition to specifying search keywords was attempted and claimed to work, but never did work. For this reason, that "feature" was not previously mentioned in the sketch specification.

During the re-implementation of the Library/Boards Manager link feature for Arduino IDE 2.x (arduino/arduino-ide#1481), the "Type" filter setting capability was finally truly implemented, and expanded to also support setting the "Topic" filter in Library Manager via the same mechanism.

This significant new functionality of the Arduino sketch format is undocumented.

What is the new behavior?

The capability released in Arduino IDE 2.0.1 is documented in the sketch specification.

Does this PR introduce a breaking change, and is titled accordingly?

No breaking change.

@per1234 per1234 added type: enhancement Proposed improvement topic: documentation Related to documentation for the project labels Oct 29, 2022
@per1234 per1234 requested a review from kittaakos October 29, 2022 03:33
@per1234 per1234 self-assigned this Oct 29, 2022
Copy link
Contributor

@kittaakos kittaakos left a comment

Choose a reason for hiding this comment

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

Maybe the docs could mention that unlike the search term in the hash, type and topic must be URL encoded:

… Manager links

Arduino IDE allows opening searches in Library Manager and Boards Manager by clicking a link with a special format in a
comment of the sketch.

During the initial implementation in Arduino IDE 1.x, support for setting the "Type" filter via the URI in addition to
specifying search keywords was attempted and claimed to work, but never did work. For this reason, it was not previously
mentioned in the sketch specification.

During the reimplementation of the Library/Boards Manager link feature for Arduino IDE 2.x, the "Type" filter setting
capability was finally truly implemented, and expanded to also support setting the "Topic" filter in Library Manager via
the same mechanism.

Now that support for the filter setting capability has been released in Arduino IDE 2.0.1, it should be documented in
the sketch specification.
@per1234
Copy link
Contributor Author

per1234 commented Oct 31, 2022

Thanks for your suggestion @kittaakos. I have now implemented it:

https://github.com/arduino/arduino-cli/compare/ca0971594241a4f0bea744a34133bb7e2a839743..1c7484ac5a152a47a67f3a21ed2b91676447e426

@kittaakos
Copy link
Contributor

Thank you for the update. It looks great. Please proceed with the merge.

@per1234 per1234 merged commit 4dcf0da into arduino:master Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: documentation Related to documentation for the project type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants