Skip to content

Conversation

@divyun
Copy link
Contributor

@divyun divyun commented Nov 5, 2025

Since 1.25.0, the dates get set to 2001-01-01T00:00:00Z, when not specified by the caller.

Fixes #35860

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Nov 5, 2025
@github-actions github-actions bot added modifies/api This PR adds API routes or modifies them modifies/go Pull requests that update Go code labels Nov 5, 2025
@divyun divyun force-pushed the fix/api/repo/contents/set-optional-dates-to-now branch from 49f4c4c to 62bff1a Compare November 5, 2025 08:56
@divyun divyun force-pushed the fix/api/repo/contents/set-optional-dates-to-now branch from 62bff1a to 19618ec Compare November 5, 2025 08:56
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Nov 5, 2025
@divyun divyun changed the title fix(api/repo/contents): set the dates to now when not specified by thee caller fix(api/repo/contents): set the dates to now when not specified by the caller Nov 5, 2025
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Nov 5, 2025
@lunny lunny added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Nov 5, 2025
@silverwind silverwind enabled auto-merge (squash) November 5, 2025 17:26
@silverwind silverwind merged commit 61e5cc1 into go-gitea:main Nov 5, 2025
25 checks passed
@GiteaBot GiteaBot added this to the 1.26.0 milestone Nov 5, 2025
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Nov 5, 2025
GiteaBot added a commit to GiteaBot/gitea that referenced this pull request Nov 6, 2025
…e caller (go-gitea#35861)

Since 1.25.0, the dates get set to `2001-01-01T00:00:00Z`, when not
specified by the caller.

Fixes go-gitea#35860

Co-authored-by: Giteabot <teabot@gitea.io>
@GiteaBot GiteaBot added the backport/done All backports for this PR have been created label Nov 6, 2025
zjjhot added a commit to zjjhot/gitea that referenced this pull request Nov 6, 2025
* giteaofficial/main:
  Remove padding override on `.ui .sha.label` (go-gitea#35864)
  fix(api/repo/contents): set the dates to now when not specified by the caller (go-gitea#35861)
  Remove `fix` Make targets (go-gitea#35868)
  Refactor ls-tree and git path related problems (go-gitea#35858)
  Fix pull description code label background (go-gitea#35865)
  Make ACME email optional (go-gitea#35849)
  Remove wrong code (go-gitea#35846)
  Fix Arch repo pacman.conf snippet (go-gitea#35825)
  Port away from `flake-utils` (go-gitea#35675)
  Update golangci-lint to v2.6.0 (go-gitea#35801)
  Add a doctor command to fix inconsistent run status (go-gitea#35840)
  Fix viewed files number is not right if not all files loaded (go-gitea#35821)
  Fix incorrect pull request counter (go-gitea#35819)
  Fix a number of `strictNullChecks`-related issues (go-gitea#35795)
  ignore .worktrees as a "special folder" (go-gitea#35835)
  upgrade go mail to 0.7.2 and fix the bug (go-gitea#35833)
silverwind pushed a commit that referenced this pull request Nov 6, 2025
…e caller (#35861) (#35874)

Backport #35861 by @divyun

Since 1.25.0, the dates get set to `2001-01-01T00:00:00Z`, when not
specified by the caller.

Fixes #35860

Co-authored-by: Divyun Raje Vaid <mail@divyun.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/done All backports for this PR have been created backport/v1.25 lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/api This PR adds API routes or modifies them modifies/go Pull requests that update Go code type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Contents POST APIs result in nil commit dates

5 participants