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

fix: replace moment() with Intl formatters #2211

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

fix: replace moment() with Intl formatters #2211

wants to merge 3 commits into from
+152 −198

Conversation

Antreesy
Copy link
Collaborator

Copied from nextcloud/talk-desktop#1107

Comparison:

Desc en_US de_DE ru_RU
moment / Intl (with 'at') 2025-02-21_12h06_49 2025-02-21_12h07_26 2025-02-21_12h07_59
Intl (with digit) / moment 2025-02-21_12h28_50 2025-02-21_12h29_21 2025-02-21_12h29_50

@Antreesy Antreesy added this to the Nextcloud 32 milestone Feb 21, 2025
@Antreesy Antreesy requested a review from ShGKme February 21, 2025 11:47
@Antreesy Antreesy self-assigned this Feb 21, 2025
ShGKme
ShGKme reviewed Feb 21, 2025
View reviewed changes
Copy link
Contributor

@ShGKme ShGKme left a comment

Choose a reason for hiding this comment

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

src/services/notificationsService.js still uses moment 👀

@Antreesy
Copy link
Collaborator Author

src/services/notificationsService.js still uses moment 👀

yes, I filtered it out somehow

@Antreesy
Copy link
Collaborator Author

/compile /

@nextcloud-command
chore(assets): Recompile assets
5e8f4d6 
Signed-off-by: nextcloud-command <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ShGKme ShGKme ShGKme approved these changes

@nickvergessen nickvergessen Awaiting requested review from nickvergessen nickvergessen is a code owner

Assignees

@Antreesy Antreesy

Labels
3. to review backport-request feature: frontend 🖌️ technical debt
Projects
None yet
Milestone
Nextcloud 32
Development

Successfully merging this pull request may close these issues.
3 participants
@Antreesy @ShGKme @nextcloud-command