Skip to content

Remove moment dependency from conversations.js

Christian Wansart requested to merge remove-moment-from-release into master

Closes #678 (closed)

What does this MR do?

Removes the only "production" dependency to momentjs which is a transitive dependency of a dev dependency. According to https://beta.foodsharing.de/assets/bundlesize.html it is not the smalles dependency and not really required.

How confident are you it won't break things if deployed?

Tested it, still works.

Links to related issues

How to test

  1. Start locally.
  2. Sign in as one user.
  3. Open a second tab (privacy mode) and log in as a different user.
  4. Write a message to the other user and check the times in the top menu.

Screenshots (if applicable)

Bildschirmfoto_2020-01-06_um_14.43.51

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s))
  • Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings
Edited by Alex

Merge request reports