-
-
Notifications
You must be signed in to change notification settings - Fork 5.9k
feat: api endpoints for projects #28111
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
base: main
Are you sure you want to change the base?
Conversation
@dineshsalunke Is the PR still WIP or is it ready for review? |
@denyskon this one is ready for review |
looks good to me! |
@dineshsalunke I see a lot of unrelated formatting changes, maybe originating from your editor formatter or similar. Could you revert them? |
@denyskon I have fixed the golines formatting and pushed. I will still check on it and get back |
@dineshsalunke Thank you, however unfortunately some are still remaining in |
Also, some methods seem to have broken when updating to main. Could you take a look please? |
@dineshsalunke I would like to review this PR for 1.25. Could you update it (and the boards pr too) to be in sync with main? |
@denyskon Surely I will update the PR |
@denyskon I have pushed some changes for now, which only include the POST methods the test's are passing for them. Work on PR was slow due to the fact that I didn't had a clear way to handle permissions for /org and /repos routes. I will look into the permissions thing now for all the POST methods and will push those changes first if the approach looks good to others then I will move with implementing rest of the routes |
…gitea into feat/api-projects
…to dineshsalunke-feat/api-projects
@dineshsalunke I’ve pushed some changes. When you have a moment, please review them. |
Fixes #14299
My earlier PR #20208 went too long and became messy, so I am closing that one and will be splitting the work in smaller chunks.
This PR includes all the endpoints for the Projects
For all the endpoints related to Boards I will be raising another PR for the same.