-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: fetch ecosystem sent invitations #123
Conversation
nishad-ayanworks
commented
Oct 5, 2023
- Worked on the GET API to fetch sent invitations to join ecosystem.
Signed-off-by: Nishad <[email protected]>
Signed-off-by: Nishad <[email protected]>
…0-ecosystem-invite Signed-off-by: Nishad <[email protected]>
Signed-off-by: Nishad <[email protected]>
import { IsOptional } from 'class-validator'; | ||
|
||
export class GetAllEcosystemInvitationsDto { | ||
@ApiProperty({ required: false }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ApiProperty({ required: false }) | |
@ApiProperty({ required: false, default: 1 }) | |
@IsOptional() | |
@IsInt() | |
@Transform(({ value }) => parseInt(value, 10)) | |
@Type(() => Number) | |
pageNumber: number; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resolved
userId: string | ||
): Promise<object> { | ||
try { | ||
const ecosystemInvitations = await this.ecosystemRepository.getInvitationsByEcosystemId(ecosystemId, pageNumber, pageSize, userId, search); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of destructuring the object in the service
file we should destruct in the repository
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resolved
Signed-off-by: Nishad <[email protected]>
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* worked on the eslint issues Signed-off-by: Nishad <[email protected]> * worked on GET API sent invitation list to join ecosystem Signed-off-by: Nishad <[email protected]> * Solved promise issue in ecosystem repository Signed-off-by: Nishad <[email protected]> * Created interface for send invitation payload Signed-off-by: Nishad <[email protected]> --------- Signed-off-by: Nishad <[email protected]>
* worked on the eslint issues Signed-off-by: Nishad <[email protected]> * worked on GET API sent invitation list to join ecosystem Signed-off-by: Nishad <[email protected]> * Solved promise issue in ecosystem repository Signed-off-by: Nishad <[email protected]> * Created interface for send invitation payload Signed-off-by: Nishad <[email protected]> --------- Signed-off-by: Nishad <[email protected]> Signed-off-by: KulkarniShashank <[email protected]>
* worked on the eslint issues Signed-off-by: Nishad <[email protected]> * worked on GET API sent invitation list to join ecosystem Signed-off-by: Nishad <[email protected]> * Solved promise issue in ecosystem repository Signed-off-by: Nishad <[email protected]> * Created interface for send invitation payload Signed-off-by: Nishad <[email protected]> --------- Signed-off-by: Nishad <[email protected]> Signed-off-by: KulkarniShashank <[email protected]>
* worked on the eslint issues Signed-off-by: Nishad <[email protected]> * worked on GET API sent invitation list to join ecosystem Signed-off-by: Nishad <[email protected]> * Solved promise issue in ecosystem repository Signed-off-by: Nishad <[email protected]> * Created interface for send invitation payload Signed-off-by: Nishad <[email protected]> --------- Signed-off-by: Nishad <[email protected]> Signed-off-by: KulkarniShashank <[email protected]>
* worked on the eslint issues Signed-off-by: Nishad <[email protected]> * worked on GET API sent invitation list to join ecosystem Signed-off-by: Nishad <[email protected]> * Solved promise issue in ecosystem repository Signed-off-by: Nishad <[email protected]> * Created interface for send invitation payload Signed-off-by: Nishad <[email protected]> --------- Signed-off-by: Nishad <[email protected]> Signed-off-by: KulkarniShashank <[email protected]>
* worked on the eslint issues Signed-off-by: Nishad <[email protected]> * worked on GET API sent invitation list to join ecosystem Signed-off-by: Nishad <[email protected]> * Solved promise issue in ecosystem repository Signed-off-by: Nishad <[email protected]> * Created interface for send invitation payload Signed-off-by: Nishad <[email protected]> --------- Signed-off-by: Nishad <[email protected]> Signed-off-by: KulkarniShashank <[email protected]>