Skip to content

Conversation

apoorvdixit88
Copy link
Contributor

@apoorvdixit88 apoorvdixit88 commented Jan 22, 2024

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

The endpoint /invite_multiple, gives support to invite more than one user at a time.

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

Currently we can only invite one user at a time.

How did you test it?

Curl for invite_multiple:

curl --location 'http://localhost:8080/user/user/invite_multiple' \
--header 'Authorization: Bearer JWT' \
--data-raw '[
    {
    "email": "[email protected]",
    "name": "25",
    "role_id": "merchant_view_only"
},
{
    "email": "[email protected]",
    "name": "25",  
    "role_id": "merchant_view_only"
},
{
    "email": "[email protected]",
    "name": "25",
    "role_id": "merchant_view_only"
}

]'
Screenshot 2024-01-23 at 4 28 32 AM

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible
  • I added a CHANGELOG entry if applicable

@apoorvdixit88 apoorvdixit88 added S-waiting-on-review Status: This PR has been implemented and needs to be reviewed M-api-contract-changes Metadata: This PR involves API contract changes R-waiting-on-L1 A-users Area: Users labels Jan 22, 2024
@apoorvdixit88 apoorvdixit88 self-assigned this Jan 22, 2024
@apoorvdixit88 apoorvdixit88 requested review from a team as code owners January 22, 2024 23:16
@apoorvdixit88 apoorvdixit88 marked this pull request as draft January 22, 2024 23:17
@apoorvdixit88 apoorvdixit88 changed the title Invite mutiple users feat(user): support multiple invites Jan 23, 2024
@apoorvdixit88 apoorvdixit88 linked an issue Jan 23, 2024 that may be closed by this pull request
@apoorvdixit88 apoorvdixit88 marked this pull request as ready for review January 23, 2024 13:20
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Jan 25, 2024
Merged via the queue into main with commit a59ac7d Jan 25, 2024
@Gnanasundari24 Gnanasundari24 deleted the invite-mutiple-users branch January 25, 2024 13:38
@SanchithHegde SanchithHegde removed S-waiting-on-review Status: This PR has been implemented and needs to be reviewed R-waiting-on-L1 labels Jan 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-users Area: Users M-api-contract-changes Metadata: This PR involves API contract changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: Make invite support email array
5 participants