TeselaGen / openVectorEditor

DEPRECATED - Teselagen's Open Source Vector/Plasmid Editor Component
https://teselagen.github.io/tg-oss/ove/#/Editor
MIT License
200 stars 72 forks source link

Broken package.json since 18.2.35 #897

Closed smeng9 closed 1 year ago

smeng9 commented 1 year ago

@tnrich

I have noticed this commit breaks the package published on npmjs https://github.com/TeselaGen/openVectorEditor/commit/70ce1c9041991b39b6207d08757310c1160adb46#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R5

The published package on npmjs https://www.npmjs.com/package/open-vector-editor/v/18.2.35?activeTab=code does not include dist folder so bundler cannot correctly resolve files

smeng9 commented 1 year ago

Hi @tnrich,

Are we planning to revert this change? Any new published package is not usable.

What is your current setup to run OVE > 18.2.35?

tnrich commented 1 year ago

Hi Smeng sorry for the breakage. I'll look into this tomorrow and fix it.

tnrich commented 1 year ago

Hi @smeng9 I think this was already fixed, see the latest version - https://www.npmjs.com/package/open-vector-editor/v/18.3.1

smeng9 commented 1 year ago

Hi @tnrich the issues persists in 18.3.1 on npmjs.

In line 8 of the package.json https://www.npmjs.com/package/open-vector-editor/v/18.3.1?activeTab=code it tries to import ./dist/index.mjs

In the published package itself https://www.npmjs.com/package/open-vector-editor/v/18.3.1?activeTab=code it has lib/ folder instead of dist/ folder.

I suggest we revert the change of line 5 and line 8 in aforementioned commit https://github.com/TeselaGen/openVectorEditor/commit/70ce1c9041991b39b6207d08757310c1160adb46#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R5

tnrich commented 1 year ago

@smeng9 thanks for the explanation, didn't realize I had forgotten those lines. They're back as they were now. Lemme know if that fixes things for you.

smeng9 commented 1 year ago

Problem is solved, Thanks!