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

Helper now useHelper #1992

Merged
merged 1 commit into from
Jun 14, 2024
Merged

Helper now useHelper #1992

merged 1 commit into from
Jun 14, 2024

Conversation

abernier
Copy link
Member

@abernier abernier commented Jun 13, 2024 •
edited
Loading

Why

because Helper and useHelper impls were so close

What

Helper component now simply takes advantage on useHelper, just passing its parent's ref to it

CleanShot 2024-06-13 at 13 33 42@2x

Checklist

  • Ready to be merged

@vercel Vercel
Copy link

vercel bot commented Jun 13, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
drei ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 13, 2024 11:30am

@codesandbox-ci CodeSandbox CI
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@abernier abernier merged commit 7645061 into master Jun 14, 2024
4 checks passed
@X-oss-byte X-oss-byte mentioned this pull request Jul 5, 2024
Open
@NOUIY NOUIY mentioned this pull request Jul 5, 2024
Open
@carltrepanier carltrepanier mentioned this pull request Jul 6, 2024
Merged
@kirankumar6676 kirankumar6676 mentioned this pull request Jul 6, 2024
Open
@David-Besse David-Besse mentioned this pull request Jul 6, 2024
Merged
@NOUIY NOUIY mentioned this pull request Jul 6, 2024
Open
@X-oss-byte X-oss-byte mentioned this pull request Jul 6, 2024
Open
@David-Besse David-Besse mentioned this pull request Jul 7, 2024
Merged
@X-oss-byte X-oss-byte mentioned this pull request Jul 7, 2024
Open
@NOUIY NOUIY mentioned this pull request Jul 8, 2024
Open
@carltrepanier carltrepanier mentioned this pull request Jul 8, 2024
Merged
@Ysrbolles Ysrbolles mentioned this pull request Jul 9, 2024
Merged
@snyk-io snyk-io bot mentioned this pull request Jul 17, 2024
Open
@hartmutobendorf hartmutobendorf mentioned this pull request Jul 17, 2024
Open
@NOUIY NOUIY mentioned this pull request Jul 17, 2024
Open
@X-oss-byte X-oss-byte mentioned this pull request Jul 17, 2024
Open
@NOUIY NOUIY mentioned this pull request Jul 18, 2024
Open
@X-oss-byte X-oss-byte mentioned this pull request Jul 18, 2024
Open
@NOUIY NOUIY mentioned this pull request Jul 20, 2024
Open
This was referenced Jul 20, 2024
Open
Open
@NOUIY NOUIY mentioned this pull request Jul 21, 2024
Open
@novikov-e novikov-e mentioned this pull request Jul 23, 2024
Open
@X-oss-byte X-oss-byte mentioned this pull request Jul 28, 2024
Open
@NOUIY NOUIY mentioned this pull request Jul 30, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@drcmda drcmda Awaiting requested review from drcmda

@CodyJasonBennett CodyJasonBennett Awaiting requested review from CodyJasonBennett

@isaac-mason isaac-mason Awaiting requested review from isaac-mason

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@abernier