Skip to content

Conversation

klakhov
Copy link
Contributor

@klakhov klakhov commented Sep 17, 2025

Motivation and context

During development of #9829 I encountered an error with setting up DatePicker. It seems antd v4 -> v5 migrated from moment to dayjs time lib. So, its impossible to init date picker with moment object.

Generally, dayjs is better than moment:

  • Smaller bundle size - Day.js is ~2KB vs Moment.js ~67KB
  • Tree-shaking friendly - Only bundled plugins that we use
  • Ant Design 5 compatibility - Native Day.js support
  • Modern API - Cleaner, immutable API
  • Better performance - Faster parsing and formatting

How has this been tested?

Checklist

  • I submit my changes into the develop branch
  • [ ] I have created a changelog fragment
  • [ ] I have updated the documentation accordingly
  • [ ] I have added tests to cover my changes
  • [ ] I have linked related issues (see GitHub docs)

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.

@codecov-commenter
Copy link

codecov-commenter commented Sep 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.19%. Comparing base (cbbdfbe) to head (f9030d5).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #9835      +/-   ##
===========================================
+ Coverage    75.48%   82.19%   +6.71%     
===========================================
  Files          410      467      +57     
  Lines        45663    48704    +3041     
  Branches      4086     4086              
===========================================
+ Hits         34469    40033    +5564     
+ Misses       11194     8671    -2523     
Components Coverage Δ
cvat-ui 77.16% <100.00%> (+<0.01%) ⬆️
cvat-server 85.99% <ø> (+11.93%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@klakhov klakhov mentioned this pull request Sep 18, 2025
6 tasks
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants