Skip to content

feat: handle issue #2#32

Open
DomeT99 wants to merge 8 commits into
DevLeonardoCommunity:mainfrom
DomeT99:main
Open

feat: handle issue #2#32
DomeT99 wants to merge 8 commits into
DevLeonardoCommunity:mainfrom
DomeT99:main

Conversation

@DomeT99

@DomeT99 DomeT99 commented Aug 17, 2024

Copy link
Copy Markdown
Contributor

✅ Set members count in the homepage (defaults to 3)
✅ Allow add/remove members in the bill page

Let me know if there are any issues :)

@netlify

netlify Bot commented Aug 17, 2024

Copy link
Copy Markdown

👷 Deploy request for billsplit-project pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 23e8667

@Balastrong Balastrong left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hey Dome, thanks for the PR! I added a few comments :)

Comment thread src/routes/index.tsx
Comment thread src/components/button/button.tsx
Comment thread src/components/member-data/member-data.tsx Outdated
Comment thread src/routes/start/[...id]/index.tsx Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants