Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: openapi for stakeholder endpoint #407

Merged
merged 54 commits into from
Jul 20, 2024
Merged

feat: openapi for stakeholder endpoint #407

merged 54 commits into from
Jul 20, 2024

Conversation

Raju-kadel-27
Copy link
Contributor

@Raju-kadel-27 Raju-kadel-27 commented Jun 29, 2024 •
edited
Loading

This pull request support following endpoints:
_stakeholder

@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 29, 2024 •
edited
Loading

Thank you for following the naming conventions for pull request titles! 🙏

@Raju-kadel-27 Raju-kadel-27 changed the title feat: openapi for stakeholder endpoint api: openapi for stakeholder endpoint Jun 29, 2024
@Raju-kadel-27 Raju-kadel-27 changed the title api: openapi for stakeholder endpoint feat: openapi for stakeholder endpoint Jun 29, 2024
@dahal dahal force-pushed the feat/openapi-for-stakeholders branch 2 times, most recently from 24a59c0 to e467ca8 Compare June 30, 2024 06:18
@Raju-kadel-27 Raju-kadel-27 force-pushed the feat/openapi-for-stakeholders branch from 0316d13 to 2f482dd Compare July 6, 2024 19:27
@Raju-kadel-27 Raju-kadel-27 marked this pull request as ready for review July 7, 2024 10:43
@Raju-kadel-27 Raju-kadel-27 requested a review from dahal July 7, 2024 10:43
@Raju-kadel-27 Raju-kadel-27 force-pushed the feat/openapi-for-stakeholders branch 2 times, most recently from f8c4368 to 2bd4cef Compare July 12, 2024 03:55
dahal
dahal requested changes Jul 17, 2024
View reviewed changes
Copy link
Contributor

@dahal dahal left a comment

Choose a reason for hiding this comment

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

Looks like update endpoint has some error. Ideally users should be able to update one or many column without having to provide all params

CleanShot 2024-07-17 at 03 17 36

@dahal dahal mentioned this pull request Jul 17, 2024
Merged
Raju-kadel-27 and others added 16 commits July 17, 2024 18:48
@Raju-kadel-27
feat: export all stakeholder routes from index
dcab3e3
@dahal @Raju-kadel-27
fix: conflict in index routes
5c769ef
@dahal @Raju-kadel-27
fix: take vs limit
529ac20
@Raju-kadel-27
feat: openapi update route for stakeholder
4c5279e
@Raju-kadel-27
feat: find many proxy for reusable cursor based pagination
edeeb8c
@Raju-kadel-27
feat: prisma-proxy for reusable cursor based pagination
6170413
@Raju-kadel-27
feat: types for reusable cursor based pagination
4ec76b5
@Raju-kadel-27
feat: openapi getMany route for stakeholder
2e398d2
@Raju-kadel-27
feat: openapi create route for stakeholder
8be410d
@Raju-kadel-27
feat: update stakeholder reusable service
4a4da6f
@Raju-kadel-27
feat: add stakeholders reusable service
05432aa
@Raju-kadel-27
feat: delete stakeholders reusable service
45378d1
@Raju-kadel-27
feat: get stakeholder by Id reusable service
42b6c30
@Raju-kadel-27
feat: get many stakeholders reusable service
e9fce0b
@Raju-kadel-27
feat: openapi route for deleting stakeholder
2b5c829
@Raju-kadel-27
feat: delete stakeholder trpc procedure
6145a8a
@Raju-kadel-27 Raju-kadel-27 force-pushed the feat/openapi-for-stakeholders branch from e869e23 to 18865e6 Compare July 17, 2024 14:40
@Raju-kadel-27
Copy link
Contributor Author

@dahal fixed those issues mentioned.

@dahal dahal assigned dahal and unassigned Raju-kadel-27 Jul 19, 2024
@dahal dahal mentioned this pull request Jul 19, 2024
Open
@dahal dahal merged commit 70d3b1a into main Jul 20, 2024
2 checks passed
@dahal dahal deleted the feat/openapi-for-stakeholders branch July 20, 2024 06:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dahal dahal dahal approved these changes

Assignees

@dahal dahal

Labels
PR: needs review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Raju-kadel-27 @dahal