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

Dynamic select changelog + docs #373

Merged

Conversation

anussel5559
Copy link
Contributor

@anussel5559 anussel5559 commented Sep 9, 2024

Description

Updates the changelog and docs to detail new dynamic variant on options in the select chain type

Known Risks

None

QA

Docs render -
image

Checklist

  • Have you read CONTRIBUTING.md already?
  • Did you update CHANGELOG.md?
    • Only user-facing changes belong in the changelog. Internal changes such as refactors should only be included if they'll impact users, e.g. via performance improvement.
  • Did you remove all TODOs?
    • If there are unresolved issues, please open a follow-on issue and link to it in a comment so future work can be tracked

@anussel5559 anussel5559 changed the base branch from master to 2.1 September 9, 2024 15:07
@anussel5559 anussel5559 force-pushed the dynamic-select-changelog-docs branch from e851316 to a8aa100 Compare September 9, 2024 15:20
CHANGELOG.md Outdated
@@ -9,6 +9,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
### Added

- Use `SLUMBER_CONFIG_PATH` to customize configuration (_not_ collection) file path [#370](https://github.com/LucasPickering/slumber/issues/370)
- Add a dynamic variant to `!select` chain type, allowing your collection to present a list of values driven from the output of another chain.
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
- Add a dynamic variant to `!select` chain type, allowing your collection to present a list of values driven from the output of another chain.
- Add a dynamic variant to `!select` chain type, allowing your collection to present a list of values driven from the output of another chain. (thanks @anussel5559)

Gotta give yourself credit!

source: !select
message: Select Fruit
options:
source: "{{chains.request_fruit}}"
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
source: "{{chains.request_fruit}}"
# Assume this respones body looks like: {"fruits": ["apple", "guava", "pear"]}
source: "{{chains.request_fruit}}"

@anussel5559 anussel5559 force-pushed the dynamic-select-changelog-docs branch from a8aa100 to d13769c Compare September 10, 2024 13:31
@LucasPickering LucasPickering merged commit bdaa7de into LucasPickering:2.1 Sep 10, 2024
15 checks passed
@anussel5559 anussel5559 deleted the dynamic-select-changelog-docs branch September 10, 2024 14:30
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.

2 participants