-
Notifications
You must be signed in to change notification settings - Fork 275
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
Fix/investment resizable layout #86
Fix/investment resizable layout #86
Conversation
Change the cookies setter key name for investment page resizable layout from `react-resizable-panels:layout` to `react-resizable-panels:layout-investment`.
Someone is attempting to deploy a commit to a Personal Account owned by @meglerhagen on Vercel. @meglerhagen first needs to authorize it. |
Hey @meglerhagen, I think this will solve the layout issue. let me know If this need changes. |
Hi @miralsuthar! Going to look at is. Is it possible that we try to take this into the turborepo? Have have refractored a lot of the code β¨ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This worked! If we could see on the other pages also that would be π―
Also, it was a format error with a space. Just remember to yarn format before the next PR β¨ |
Got it, thank you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, merging now.
Description
This PR fixes the bug.
What type of PR is this? (check all applicable)
Related Tickets & Documents
Please use this format link issue numbers: Fixes #42
Mobile & Desktop Screenshots/Recordings
Steps to QA
Added to documentation?
[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?