feat(api): Workspace-membership invitationAccepted included

[csehatt741]

User description

Description

WorkspaceMember.invitationAccepted returned in /api/workspace-membership/{workspaceSlug}/members

Fixes #406

Screenshots of relevant screens

Developer's checklist

• My PR follows the style guidelines of this project
• I have performed a self-check on my work

If changes are made in the code:

• I have followed the coding guidelines
• My changes in code generate no new warnings
• My changes are breaking another fix/feature of the project
• I have added test cases to show that my feature works
• I have added relevant screenshots in my PR
• There are no UI/UX issues

Documentation Update

• This PR requires an update to the documentation at docs.keyshade.xyz
• I have made the necessary updates to the documentation, or no documentation changes are required.

---

PR Type

Enhancement

---

Description

• Added invitationAccepted field to workspace membership API response.

• Enhanced /api/workspace/{workspaceId}/members endpoint to include membership status.

• Updated service logic to fetch and return invitation status.

• Improved API functionality for better workspace member management.

---

Changes walkthrough 📝

	Relevant files
Enhancement	workspace-membership.service.ts Include invitation status in workspace membership API        apps/api/src/workspace-membership/service/workspace-membership.service.ts Added invitationAccepted field to the API response. Enhanced query to include invitation status in the returned data. +2/-1

---

Need help?

Type /help how to ... in the comments thread for any questions about Qodo Merge usage.

Check out the documentation for more information.

[codiumai-pr-agent-free]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis 🔶 406 - Partially compliant Compliant requirements: Add 'active' field to endpoint response (implemented as 'invitationAccepted') Update service function to fetch invitation status Non-compliant requirements: Include tests for the changes Requires further human verification: Verify that invitationAccepted field correctly reflects the invitation status in API responses

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Missing Tests The PR adds new functionality but lacks test coverage to verify the invitation status is correctly returned invitationAccepted: true

[codiumai-pr-agent-free]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Score
General	Add null check for data integrity Consider adding a null check for the invitationAccepted field in the query to handle cases where the invitation status might not be set. apps/api/src/workspace-membership/service/workspace-membership.service.ts [520] -invitationAccepted: true +invitationAccepted: { + select: true, + where: { + NOT: null + } +} Apply this suggestion Suggestion importance[1-10]: 3 Why: While adding a null check could provide additional data validation, the current implementation using 'true' is a standard Prisma select statement and is sufficient for most use cases. The suggestion adds unnecessary complexity without significant benefit.	3

[rajdip-b]

Can't always be true. This should come from the invitation status field in workspace membership model

[csehatt741]

invitationAccepted: true is how Prisma configured to include a specific field in the return type.

[rajdip-b]

I need to get my eyes checked. Sorry :)

[rajdip-b]

Just the tests would work

[rajdip-b]

Also, could you add relevant tests for this?

[csehatt741]

This week I'm off. Next week I'll sort out this and all other issues too.

[rajdip-b]

Thanks!