Skip to content

fix: add example for destructure property ‘basename’ of s.useContext issue #284

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

VenkatS1208
Copy link

@VenkatS1208 VenkatS1208 commented Apr 2, 2025

Fixes #283

Copy link
Collaborator

@gioboa gioboa left a comment

Choose a reason for hiding this comment

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

Thanks for your commitment @VenkatS1208
I will look at this in few hours.
I think it's better to include the fix and not all the other files

@gioboa gioboa changed the title mfe issue fix: fix up destructure property ‘basename’ of s.useContext(…) Apr 2, 2025
@gioboa
Copy link
Collaborator

gioboa commented Apr 4, 2025

Thanks @VenkatS1208 for this example.
Have you found a solution for this issue?

@gioboa gioboa changed the title fix: fix up destructure property ‘basename’ of s.useContext(…) fix: add example for destructure property ‘basename’ of s.useContext issue Apr 4, 2025
@VenkatS1208
Copy link
Author

VenkatS1208 commented Apr 5, 2025

Thanks @VenkatS1208 for this example. Have you found a solution for this issue?

Hi @gioboa , tried with couple of changes still same issue

Copy link
Collaborator

@gioboa gioboa left a comment

Choose a reason for hiding this comment

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

I see. Right now I don't have the possibility to check it but we can leave it here for reference.
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants