[bugfix] './locale' path not found in moment/src/lib/locale/locales.js

[ApacheEx]

seems root of issue was introduced here: 9ce89e7

So, this PR reverts it and fixes the following issues:
#4505, #5472, #5497, #5501, #5500, #5378, #5495, #5506, #5508

[jsf-clabot]

All committers have signed the CLA.

[coveralls]

Coverage decreased (-0.04%) to 88.425% when pulling 7df7e6b on ApacheEx:patch-1 into 475a1e0 on moment:develop.

[ApacheEx]

also, as possible fix:

replace "module": "./src/moment.js" to "module": "./moment.js"
but don't know exactly if this "module" is really needed at all and what was the reason to introduce it.

[J1445]

also, as possible fix:

replace "module": "./src/moment.js" to "module": "./moment.js"
but don't know exactly if this "module" is really needed at all and what was the reason to introduce it.

Can confirm this fixes #5495 the issue and you are the MVP here.

Also commit description is not clear in 9ce89e7

[nicolashenry]

FYI "module" is supposed to be an ecmascript module and "./moment.js" is a commonjs module. (https://webpack.js.org/guides/author-libraries/#final-steps)

[BuangHosen]

also, as possible fix:

replace "module": "./src/moment.js" to "module": "./moment.js"
but don't know exactly if this "module" is really needed at all and what was the reason to introduce it.

thanks sir, you save my day 👍

[FredrikSeidl]

Can confirm that this resolves it for us. Can this be merged and released on NPM soon?

[ApacheEx]

ok guys, I see it's fixed in 2.25.2.

So, I'm closing this.

[ApacheEx]

"module" property was removed in 2.25.3 (0c709ba)

So, it was the right solution.