[2.x] Inertia: Remove moment.js dependency

[claudiodekker]

While cleaning up the app.js, I noticed that the moment library was included, but was only being used once.

Because Inertia gets it's state from the back-end, we can instead let Carbon do this conversion for us and pass it through to the front-end and remove the (fairly large) moment dependency entirely.

[reinink]

Love this change, simply because Moment.js is huge. I personally removed it from my own SaaS app recently for the same reason. Read more about the issues with Momment here.

[taylorotwell]

One benefit of using Moment is the ability to use the user's local browser timezone when rendering / manipulating dates. Does this still display accurately?

[reinink]

@taylorotwell Great point, I don't think this does. Let us investigate. 👍

[claudiodekker]

@taylorotwell I don't think it'd matter in this case.. The server stores it's server-side timestamp in the database, and on retrieval it'd diff against the server-side time at that point, so wouldn't the outcome always be identical regardless?:

Or am I overlooking something / misunderstanding?

[reinink]

@claudiodekker Yup, you're right. Since the diff is done between two UTC dates, you'll end up with identical results either way. Good catch. 👍