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

Show 'slots' syntax #5

Merged
merged 3 commits into from
Feb 27, 2022
Merged

Show 'slots' syntax #5

merged 3 commits into from
Feb 27, 2022

Conversation

ZacButko
Copy link
Contributor

@ZacButko ZacButko commented Feb 3, 2022

Updated extensions declaration to show 'slots' syntax. More clear how to use slot vs slots.
Let me know if you guys think this is silly, but this took me a while to track down why it wasn't working 😅

I assumed from http://o3-dev.docs.openmrs.org/#/main/extensions page hint "The extension names the slot in its definition, under slot[s]" that the syntax would be slot: "Boxes" singular and slot: ["Boxes"] plural, not slots: ["Boxes"]. In retrospect wording on that page makes sense, but it would be good to show example in the example code

Updated extensions declaration to show 'slots' syntax. More clear how to use slot vs slots.
Let me know if you guys think this is silly, but this took me a while to track down why it wasn't working 😅
src/index.ts Outdated Show resolved Hide resolved
Co-authored-by: Brandon Istenes <bistenes@gmail.com>
Copy link
Collaborator

@brandones brandones left a comment

Choose a reason for hiding this comment

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

Thanks @ZacButko . Could you fix the build?

@ZacButko ZacButko merged commit 0938ff5 into master Feb 27, 2022
@ZacButko ZacButko deleted the update/extensions-example branch February 27, 2022 06:51
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.

None yet

2 participants