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.

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: added /stakeholder/stakeholder_id endpoint #377

Merged

Conversation

XLPeng57
Copy link

@XLPeng57 XLPeng57 commented Mar 10, 2024

Added endpoint for #372

fixed: #372

@dadiorchen
Copy link
Contributor

When you use: fixed: #xxx it will link the PR with issue, and automatically close issue when pr get merged

@dadiorchen
Copy link
Contributor

The unit test looks great.

We didn't set up a good base for this repo, so we just do unit test to cover the repository class, ideally we should use integration test to cover code crossing files.

So did you try to run the server and tested the endpoint?

@dadiorchen dadiorchen merged commit fa194cb into Greenstand:main Mar 23, 2024
1 check passed
@dadiorchen
Copy link
Contributor

Great job, thank you for your contribution!

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.

Create stakeholder/{stakehoulder_it} api
2 participants