-
Notifications
You must be signed in to change notification settings - Fork 2
Move 'New Message' button from mailbox/page.tsx to members/page.tsx #40
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
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
✨ CodeStory generated PR Summary: Install extension at link for in editor experience Open in editor [🔗] copy paste in your browser address bar 🤖 (564e982) 1: The 2: The If you found this useful, please give me feedback by liking 👍 this comment, or 👎 if this wans't useful to you. I learn from my mistakes 🎓 I am a AI bot and might make mistakes 🫤, please give feedback to the devs at founders@codestory.ai or on slack |
|
ReactServerComponentsError: You're importing a component that needs useState. It only works in a Client Component but none of its parents are marked with "use client", so they're Server Components by default. ,-[/Users/alanaanderson/Developer/rebase/app/members/page.tsx:1:1] Maybe one of these should be marked as a client entry with "use client": |
Hi @alanagoyal, I decided to make the following changes:
|
|
./app/members/page.tsx Caused by: |
Hi @alanagoyal, I decided to make the following changes:
|
Description
This PR moves the 'New Message' button from the mailbox/page.tsx file to the members/page.tsx file next to the 'Add Member' button. It also ensures that the button's event handlers and functions are properly moved and still function correctly.
Summary of Changes
Please review and merge this PR. Thank you!
Fixes #38.
To checkout this PR branch, run the following command in your terminal:
🎉 Latest improvements to Sweep:
💡 To get Sweep to edit this pull request, you can: