dymo-connect-framework icon indicating copy to clipboard operation
dymo-connect-framework copied to clipboard

Project re-write to use node instead of just compiled js

Open zach2825 opened this issue 3 years ago • 8 comments

Converting plain js into a modern node syntax that uses promises for all API calls.

I've done that so that everything is predictable and we can add timeouts to API calls and have better control over the code.

If anybody is interested let me know. If I am able to move forward with my project I'd be happy to invite people to it

I've already broken out the 41,000 line js into separate code and using node packages when possible. I was able to remove all the XHR and goog stuff which simplifies a ton. I've also tried fixing as many spelling mistakes as possible.

image

zach2825 avatar Feb 17 '22 22:02 zach2825

🙋‍♂️ Interested!

blester-mvc avatar Feb 17 '22 23:02 blester-mvc

I've only copied most of the functions into my project, removed the goog code, and used a package, I'm using Axios for API communication and I have getSettings and setSettings functions. I've done basic testing but I spent the day converting the framework.

I need help:

  • adding compile code to build the JS like this repo that can be included in browser script
  • building sample code
  • making sure what I have is complete.

I ask that the working branch is develop but please feel free to make it yours an contribute if possible https://github.com/1to1plus/dymo-js-sdk

zach2825 avatar Feb 18 '22 13:02 zach2825

removed the google code

Wohoo!! I mentioned the google code in issue #49 .

Did you make a pull request? It might be better to fork things as @dymosoftware just seems to wanna turn the page and not listen to feedback.

This library needs proper project management!

timint avatar Sep 03 '22 21:09 timint

I did fork the repo and do a bunch of things over there. I did my best to update everything to factor out the google libs and make all actions execute using promises @timint

zach2825 avatar Sep 06 '22 20:09 zach2825

@zach2825 Well done. It's the right thing to do. This library is a hundred times bigger than it needs to be. I have tried reaching management on Dymo.

timint avatar Sep 07 '22 08:09 timint

Please feel free to check it out. It may not be perfect, please keep that in mind I didn't have much time to update it. We've been using the new code for like a year without problems. The link to the forked repo is above. I welcome any help. Good luck @timint

zach2825 avatar Sep 07 '22 12:09 zach2825

@dymosoftware 554KB of lib for some local http call, isnt it mad? Is there any hope for a refresh?

LOL zebra vs dymo size comparison (more or less the same lib philosophy) image

Roman991 avatar Jan 22 '23 17:01 Roman991

@Roman991 This is just the tip of the ice berg. Looking at the code it's obvious they don't know what they are doing. Looking at this repo it's obvious they don't know how to maintain a repo neither.

I tried to reach management to let them know how bad things are. But they actively rejected me. Management is probably just as bad.

timint avatar Jan 22 '23 18:01 timint