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

Wrong meta info at issueComments on github migration #9741

Closed
6543 opened this issue Jan 13, 2020 · 3 comments · Fixed by #9744
Closed

Wrong meta info at issueComments on github migration #9741

6543 opened this issue Jan 13, 2020 · 3 comments · Fixed by #9744
Labels
skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. type/bug
Milestone
1.11.0

Comments

@6543
Copy link
Member

6543 commented Jan 13, 2020

@lunny if I migrate a repo with issues:

comments are generated wich have the correct creation timestamp but update-timestamp is "1970-01-01T01:00:00+01:00"
@bagasme
Copy link
Contributor

bagasme commented Jan 13, 2020

@6543 Beginning Unix time epoch?

@lunny lunny added the type/bug label Jan 13, 2020
@6543
Copy link
Member Author

6543 commented Jan 13, 2020

exactly -> migration simply dont set time on updated attribute

@6543
Copy link
Member Author

6543 commented Jan 13, 2020

updated should never be less than created 😅 ...

@lunny lunny mentioned this issue Jan 13, 2020
Merged
@lunny lunny added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label Jan 13, 2020
@lunny lunny added this to the 1.11.0 milestone Jan 13, 2020
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. type/bug
Projects
None yet
Milestone
1.11.0
Development

Successfully merging a pull request may close this issue.

Fix missing updated time on migrated issues and comments lunny/gitea
3 participants
@lunny @6543 @bagasme