Skip to content
This repository was archived by the owner on Jun 8, 2019. It is now read-only.

Updates to swagger documentation #77

Merged
merged 1 commit into from
Nov 12, 2017
Merged

Conversation

ethantkoenig
Copy link
Member

This is part of a larger updating/refactoring endeavor that will fix go-gitea/gitea#2635 and other problems with our swagger docs. See https://github.com/ethantkoenig/gitea/tree/swagger if you're curious.

This PR has no functional change.

@lafriks
Copy link
Member

lafriks commented Nov 9, 2017

LGTM

@appleboy
Copy link
Member

LGTM

@lafriks lafriks merged commit 1da52cf into go-gitea:master Nov 12, 2017
@lunny
Copy link
Member

lunny commented Nov 12, 2017

Please send a PR to update this to gitea vendor.

CreatedAt time.Time `json:"created_at"`
PublishedAt time.Time `json:"published_at"`
Publisher *User `json:"author"`
Attachments []*Attachment `json:"assets"`
Copy link
Member

Choose a reason for hiding this comment

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

Yes, attachements should not be removed here.

Copy link
Member Author

Choose a reason for hiding this comment

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

See discussion in #83

Copy link
Member Author

Choose a reason for hiding this comment

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

I probably should have commented in this PR about why I removed it, so apologies for miscommunication on my part.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Swagger errors
5 participants