Skip to content

fix(invite): allow anyone with access to the workspace to invite others, remove invite from dev environment#379

Merged
waleedlatif1 merged 3 commits intomainfrom
fix/invite
May 19, 2025
Merged

fix(invite): allow anyone with access to the workspace to invite others, remove invite from dev environment#379
waleedlatif1 merged 3 commits intomainfrom
fix/invite

Conversation

@waleedlatif1
Copy link
Collaborator

Description

Allow anyone with access to the workspace to invite others, remove invite from dev environment

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works
  • All tests pass locally and in CI (bun run test)
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • I have updated version numbers as needed (if needed)
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Security Considerations:

  • My changes do not introduce any new security vulnerabilities
  • I have considered the security implications of my changes

@vercel
Copy link

vercel bot commented May 19, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sim ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 19, 2025 10:46pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Skipped (Inspect) May 19, 2025 10:46pm

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This PR modifies the workspace invitation system to allow any workspace member to invite others and removes the invite feature from development environments.

  • Removed owner-only restriction in /apps/sim/app/api/workspaces/invitations/route.ts, allowing any workspace member to send invitations
  • Added environment-based conditional rendering in /apps/sim/app/w/components/sidebar/sidebar.tsx to hide invite functionality in dev mode
  • Implemented new table-based UI in /apps/sim/app/w/components/sidebar/components/invite-modal/invites-sent/invites-sent.tsx for better invitation status display
  • Updated /apps/sim/app/w/components/sidebar/components/invite-modal/invite-modal.tsx with improved email validation and error handling

4 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile

@waleedlatif1 waleedlatif1 merged commit 2a4590e into main May 19, 2025
6 checks passed
@waleedlatif1 waleedlatif1 deleted the fix/invite branch May 19, 2025 22:52
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.

1 participant