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

chore: replace react-portal with react-use-portal #5383

Conversation

Jay-Topher
Copy link
Contributor

Done

  • Replaced instance of react-portal with react-use-portal

QA steps

  • Ensure that app works as expected
  • Ensure that the "Create new tag" dialog in the controller configuration page opens as expected

Fixes

Fixes: 1599

Screenshots

image

Notes

@webteam-app
Copy link

@Jay-Topher Jay-Topher force-pushed the MAASENG-1599-replace-react-portal-with-useportal branch from 6a733f9 to 1b41958 Compare April 2, 2024 16:22
@Jay-Topher Jay-Topher force-pushed the MAASENG-1599-replace-react-portal-with-useportal branch from 1b41958 to f015b25 Compare April 4, 2024 07:54
@Jay-Topher Jay-Topher force-pushed the MAASENG-1599-replace-react-portal-with-useportal branch 2 times, most recently from 19561c2 to 85e261d Compare April 8, 2024 11:07
Copy link
Contributor

@petermakowski petermakowski left a comment

Choose a reason for hiding this comment

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

Like I explained in the previous review, I'd recommend to use a curried function here. Accepting my inline review suggestions should make this work.

src/app/base/components/TagSelector/TagSelector.tsx Outdated Show resolved Hide resolved
src/app/base/components/TagSelector/TagSelector.tsx Outdated Show resolved Hide resolved
src/app/base/components/TagSelector/TagSelector.test.tsx Outdated Show resolved Hide resolved
@Jay-Topher Jay-Topher force-pushed the MAASENG-1599-replace-react-portal-with-useportal branch from bb09446 to e40f32f Compare April 8, 2024 13:36
@Jay-Topher Jay-Topher force-pushed the MAASENG-1599-replace-react-portal-with-useportal branch from e40f32f to c7a0663 Compare April 8, 2024 14:22
@petermakowski petermakowski force-pushed the MAASENG-1599-replace-react-portal-with-useportal branch from 54bbd11 to 637b7b4 Compare April 9, 2024 12:48
@petermakowski petermakowski enabled auto-merge (squash) April 9, 2024 12:49
@petermakowski petermakowski merged commit 0aae3f2 into canonical:main Apr 9, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants