Skip to content

Guide Fixes from User Feedback # 1#5128

Merged
cielf merged 1 commit into
rubyforgood:mainfrom
cielf:Guide_fixes_from_user_input_1
Apr 4, 2025
Merged

Guide Fixes from User Feedback # 1#5128
cielf merged 1 commit into
rubyforgood:mainfrom
cielf:Guide_fixes_from_user_input_1

Conversation

@cielf
Copy link
Copy Markdown
Collaborator

@cielf cielf commented Apr 1, 2025

Description

Minor guide fixes from user feedback
(Our beta tester for packs is also taking a a look at the guide. Hooray!)

1/ Substituting "--" for "|" wherever we are describing a chain of menu clicks, as the pipe indicates tabular format in the gitbook'ed version.
2/ Typo(s)

Type of change

  • Documentation update

@cielf
Copy link
Copy Markdown
Collaborator Author

cielf commented Apr 1, 2025

@awwaiid, @dorner -- I'd like to get these in this week - but let's hold off approving/merging until Friday, and I'll add anything else our beta tester spots in the meantime. If that's workable.

@cielf cielf requested review from awwaiid and dorner April 1, 2025 01:36
Except for date range, all the filters are specified by picking from a drop-down list as follows:
- Item: all of your bank's Items. This will filter to only the distributions that contain that Item.
- Item Category: Item Categories (as specified in [Items & Inventory | Item Categories](inventory_items.md)). This will filter the list to the distributions that contain items that are in the chosen item category.
- Item Category: Item Categories (as specified in [Items & Inventory -- Item Categories](inventory_items.md)). This will filter the list to the distributions that contain items that are in the chosen item category.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Yes, I agree with this editorial/stylistic change!

@awwaiid awwaiid self-requested a review April 1, 2025 11:46
@awwaiid
Copy link
Copy Markdown
Collaborator

awwaiid commented Apr 1, 2025

Looks good so farm

@cielf cielf merged commit 91ebd12 into rubyforgood:main Apr 4, 2025
11 of 12 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 6, 2025

@cielf: Your PR Guide Fixes from User Feedback # 1 is part of today's Human Essentials production release: 2025.04.06.
Thank you very much for your contribution!

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