Skip to content

Remove jstz usage from server - #36000

Merged
szaimen merged 1 commit into
masterfrom
remove-jstz
Jan 4, 2023
Merged

Remove jstz usage from server#36000
szaimen merged 1 commit into
masterfrom
remove-jstz

Conversation

@tcitworld

Copy link
Copy Markdown
Member

Every proper browser has this kind of data now, with more up-to-date data (from ICU) than what's inside this package, which hasn't been updated in 3 years.

pellepim/jstimezonedetect#6
pellepim/jstimezonedetect#9

12Ko JS files less \o/

@tcitworldtcitworld added this to the Nextcloud 26 milestone Jan 4, 2023
@tcitworld
tcitworld requested review from a team, ChristophWurst, Pytal, artonge and szaimen and removed request for a teamJanuary 4, 2023 16:39
@tcitworldtcitworld self-assigned this Jan 4, 2023
tcitworld added a commit that referenced this pull request Jan 4, 2023
Follow-up to #36000
Signed-off-by: Thomas Citharel <tcit@tcit.fr>
tcitworld added a commit that referenced this pull request Jan 4, 2023
Follow-up to #36000
Signed-off-by: Thomas Citharel <tcit@tcit.fr>
Every proper browser has this kind of data now, with more up-to-date
data than what's inside this package, which hasn't been updated in 3
years.
pellepim/jstimezonedetect#6pellepim/jstimezonedetect#9
Signed-off-by: Thomas Citharel <tcit@tcit.fr>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@szaimenszaimen added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 4, 2023
@szaimen
szaimen enabled auto-merge January 4, 2023 19:17
@szaimen
szaimen disabled auto-merge January 4, 2023 21:40
@szaimen
szaimen merged commit fec5735 into masterJan 4, 2023
@szaimen
szaimen deleted the remove-jstz branch January 4, 2023 21:40
akhil1508 pushed a commit to e-foundation/server that referenced this pull request Sep 28, 2023
Follow-up to nextcloud#36000
Signed-off-by: Thomas Citharel <tcit@tcit.fr>
Signed-off-by: Akhil <akhil@e.email>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishdependenciesjavascripttechnical debt🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@tcitworld@artonge@Pytal@szaimen