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

[Feature] [API] GET commit content #10921

Closed
6543 opened this issue Apr 1, 2020 · 6 comments · Fixed by #17095
Closed

[Feature] [API] GET commit content #10921

6543 opened this issue Apr 1, 2020 · 6 comments · Fixed by #17095
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented modifies/api This PR adds API routes or modifies them

Comments

@6543
Copy link
Member

6543 commented Apr 1, 2020

this api does not exist on Github and its needed for web clients to view commit content (they usuale dont have git installed 😉 )
@6543
Copy link
Member Author

6543 commented Apr 1, 2020 •
edited
Loading

@guillep2k I'll take this too (hope it can go into v1.12.0 ) ...

UseCase: https://gitea.com/gitnex/GitNex/issues/335

@guillep2k guillep2k added the modifies/api This PR adds API routes or modifies them label Apr 2, 2020
@6543
Copy link
Member Author

6543 commented Apr 2, 2020

similar output format as #10933 introduce

api endpint proposal: GET /repos/:owner/:repo/commits/:ref/files

@6543 6543 mentioned this issue Apr 29, 2020
Closed
@6543
Copy link
Member Author

6543 commented Apr 29, 2020 •
edited
Loading

depending on how #11187 make progress I'll start implementing it or waiting for this pull ...

@stale
Copy link

stale bot commented Jun 28, 2020

This issue has been automatically marked as stale because it has not had recent activity. I am here to help clear issues left open even if solved or waiting for more insight. This issue will be closed if no further activity occurs during the next 2 weeks. If the issue is still valid just add a comment to keep it alive. Thank you for your contributions.

@stale stale bot added the issue/stale label Jun 28, 2020
@6543
Copy link
Member Author

6543 commented Jun 28, 2020

no stale #11187 is kind a stale at the moment :/

@stale stale bot removed the issue/stale label Jun 28, 2020
@lunny lunny added the issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented label Jul 7, 2020
@6543 6543 removed their assignment Jul 27, 2020
@6543
Copy link
Member Author

6543 commented Jul 27, 2020

de-assigne me so others can pic up if they have more time than me at the moment

throu It wont be forgotten by me ;)

@6543 6543 mentioned this issue Oct 2, 2020
Closed
7 tasks
@qwerty287 qwerty287 mentioned this issue Sep 19, 2021
Merged
@go-gitea go-gitea locked and limited conversation to collaborators Oct 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented modifies/api This PR adds API routes or modifies them
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add API to get commit diff/patch
3 participants
@lunny @guillep2k @6543