Skip to content

feat: ✨ remove member availability from a meeting#525

Merged
ethancha0 merged 5 commits into
mainfrom
et-remove-member-availability-meeting
May 21, 2026
Merged

feat: ✨ remove member availability from a meeting#525
ethancha0 merged 5 commits into
mainfrom
et-remove-member-availability-meeting

Conversation

@realethantran
Copy link
Copy Markdown
Contributor

Description

This PR adds the ability for the host of a meeting to remove a member from the meeting. This action both removes the member from the meeting and their availability.

Recording/Screenshots

Before

After

Test Plan

Issues

@realethantran realethantran temporarily deployed to staging-525 May 18, 2026 20:38 — with GitHub Actions Inactive
@github-actions github-actions Bot temporarily deployed to staging-525 May 18, 2026 20:39 Inactive
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 4 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread src/components/availability/group-responses.tsx Outdated
@ethancha0
Copy link
Copy Markdown
Collaborator

image

issue: can we have the close icon persist on mobile view, but keep the same hover logic on desktop

@ethancha0
Copy link
Copy Markdown
Collaborator

Screen.Recording.2026-05-19.at.9.51.26.AM.mov

issue: flickering seems to be a product of adding the close icon with a long name. This should be placed in the following row

@realethantran
Copy link
Copy Markdown
Contributor Author

Screen.Recording.2026-05-19.at.9.51.26.AM.mov
issue: flickering seems to be a product of adding the close icon with a long name. This should be placed in the following row

I will take a look!

@ethancha0
Copy link
Copy Markdown
Collaborator

Screen.Recording.2026-05-19.at.9.51.26.AM.mov
issue: flickering seems to be a product of adding the close icon with a long name. This should be placed in the following row

I will take a look!

@ethancha0 ethancha0 closed this May 19, 2026
@ethancha0 ethancha0 temporarily deployed to staging-525 May 19, 2026 21:58 — with GitHub Actions Inactive
@ethancha0 ethancha0 reopened this May 19, 2026
@realethantran realethantran temporarily deployed to staging-525 May 21, 2026 04:08 — with GitHub Actions Inactive
@github-actions github-actions Bot temporarily deployed to staging-525 May 21, 2026 04:08 Inactive
@realethantran realethantran temporarily deployed to staging-525 May 21, 2026 04:19 — with GitHub Actions Inactive
@github-actions github-actions Bot temporarily deployed to staging-525 May 21, 2026 04:19 Inactive
@realethantran realethantran requested a review from ethancha0 May 21, 2026 04:20
@ethancha0 ethancha0 temporarily deployed to staging-525 May 21, 2026 05:51 — with GitHub Actions Inactive
@github-actions github-actions Bot temporarily deployed to staging-525 May 21, 2026 05:52 Inactive
Copy link
Copy Markdown
Collaborator

@ethancha0 ethancha0 left a comment

Choose a reason for hiding this comment

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

YAYYY GOOD JOB ETHAN

@ethancha0 ethancha0 merged commit a06c0dd into main May 21, 2026
2 checks passed
@realethantran
Copy link
Copy Markdown
Contributor Author

YAYYY GOOD JOB ETHAN

TYY ETHAN!

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.

Remove member + availability for meeting

2 participants