Port project to Clasp - Githubissues

ziolko / spreadapi

Free REST API for Google Spreadsheet

https://spreadapi.roombelt.com

Other

87 stars 13 forks source link

Port project to Clasp #8

Closed ChakshuGautam closed 2 years ago

ChakshuGautam commented 3 years ago

Hi,

I love your project and would like to port this to Clasp. I can either create a PR in this repo or I can fork and maintain a different repo.

I am not sure if this is the direction you would want to take the project so please let me know if this can be a possibility.

Thanks.

ziolko commented 3 years ago

Hi. I am glad you are open to help! What I like about the current setup is that everybody can easily review and change the source code without setting up anything on their side. With Clasp, What would one need to do to modify the code starting without any development environment?

ChakshuGautam commented 3 years ago

It would output a similar code to the *.min.js file. The changes would enable the following

Everything would be in Typescript and changes need to happen in the core code. It is possible to distribute the controllers in 4 files so that shouldn't be much if we provide docs.
Testing can happen through Jest for anything other than non-mockable GAS code.
Deployment lifecycle would be managed in a similar way as git. It will be converted to gs files and deployed directly through the deploy command to the doc using the doc id.

ziolko commented 3 years ago

I see you have quite ambitious goals in mind :) I'd rather keep this repository simple so that everybody can review and customize the code without installing additional tools.

I think the way to go is to fork this repository and introduce suggested changes there. If this ends up well I will have no issues with linking the fork from here.

ChakshuGautam commented 3 years ago

Awesome. Will start the port sometime this weekend. Will keep you posted. Thanks for allowing me to do so. :)

ziolko commented 2 years ago

Closing due to inactivity.