Use default server time zone if non provided #2333

christianlupus · 2024-05-26T17:58:55Z

Topic and Scope

This is mainly a quick fix as the timezone information is not present in the database ATM. Instead, the server time zone is used and assumed stable.

There are some formal requirements that should be satisfied. Please mark those by checking the corresponding box.

I did check that the app can still be opened and does not throw any browser logs
I created tests for newly added PHP code (check this if no PHP changes were made)
I updated the OpenAPI specs and added an entry to the API changelog (check if API was not modified)
I notified the matrix channel if I introduced an API change

github-actions · 2024-05-26T18:06:23Z

12 files 584 suites 1m 33s ⏱️
573 tests 573 ✅ 0 💤 0 ❌
2 292 runs 2 291 ✅ 1 💤 0 ❌

Results for commit 6ac73b4.

♻️ This comment has been updated with latest results.

christianlupus · 2024-05-26T18:12:01Z

/approve

nextcloud-cookbook-bot

Automatically approved by GitHub action on behalf of /approve

Signed-off-by: Christian Wolf <github@christianwolf.email>

christianlupus marked this pull request as ready for review May 26, 2024 18:04

nextcloud-cookbook-bot approved these changes May 26, 2024

View reviewed changes

christianlupus force-pushed the fix/1456-timezone-in-recipe-stub branch from 75d0844 to 2303885 Compare May 29, 2024 19:37

christianlupus added 2 commits May 30, 2024 10:15

Use default server time zone if non provided

49633ab

Signed-off-by: Christian Wolf <github@christianwolf.email>

Update Changelog

6ac73b4

Signed-off-by: Christian Wolf <github@christianwolf.email>

christianlupus force-pushed the fix/1456-timezone-in-recipe-stub branch from 2303885 to 6ac73b4 Compare May 30, 2024 08:20

christianlupus merged commit 49a49e5 into master May 30, 2024
31 checks passed

christianlupus deleted the fix/1456-timezone-in-recipe-stub branch May 30, 2024 08:36