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.

Sign up for GitHub

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

Jump to bottom

Fixed issue #1792: Used of modifier instead of component. #1793

Merged
merged 1 commit into from
Jul 12, 2024
Merged

Fixed issue #1792: Used of modifier instead of component. #1793

merged 1 commit into from
Jul 12, 2024

Conversation

bitxplora
Copy link
Contributor

@bitxplora bitxplora commented Jul 12, 2024 •
edited by NullVoxPopuli
Loading

This issue was caused by a mix-up of modifier for component, where modifier was used in the prose title, but in the body it was component. This happened in the explanation of component partial application.

Resolves #1792

component is used for partial application of arguments to modifiers.

This can be useful for pre-wiring arguments to complex components

Summary

copilot:summary

Details

@bitxplora
Fixed issue NullVoxPopuli#1792: Used of modifier instead of component.
b5eb440 
This issue was caused by a mixed up of modifier for component, where
modifier was used in the prose title, but in the body it was component.
This happened in the explanation of component partial application.
@stackblitz StackBlitz
Copy link

stackblitz bot commented Jul 12, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

NullVoxPopuli
NullVoxPopuli approved these changes Jul 12, 2024
View reviewed changes
Copy link
Owner

@NullVoxPopuli NullVoxPopuli left a comment

Choose a reason for hiding this comment

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

Thank you! I must have made a copy paste mistake

@NullVoxPopuli NullVoxPopuli merged commit d795768 into NullVoxPopuli:main Jul 12, 2024
8 checks passed
@github-actions github-actions bot mentioned this pull request Jul 12, 2024
Merged
@NullVoxPopuli NullVoxPopuli added the documentation Improvements or additions to documentation label Jul 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NullVoxPopuli NullVoxPopuli NullVoxPopuli approved these changes

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Mixed up: used modifier instead of component
2 participants
@bitxplora @NullVoxPopuli