search

news-catalyst / google-app-scripts

a collection of scripts for tinynews google docs add-ons
https://script.google.com/a/newscatalyst.org/d/1ILURq69o3cYUy6k1n1X6HwxdMfl9xWNhILYuZxgLfeblb3IR15WCMZSj/edit
1 stars 1 forks source link

Update "admin tools" config form #414

Closed jacqui closed 2 years ago

jacqui commented 2 years ago

Closes #413

The admin tools now only includes the settings required for the publishing sidebar to function.

TylerFisher commented 2 years ago

What do we still use the content API/token for? Doesn't all the Hasura interaction happen at the next.js level now?

jacqui commented 2 years ago

What do we still use the content API/token for? Doesn't all the Hasura interaction happen at the next.js level now?

I just did a pass of all of the code for references to fetchGraphQL, which is the function that was using those two values. All interactions with Hasura happen through the front-end next.js API, so I've gone through and cleaned up all of the code that was relying on the old Hasura API access. Tested both Admin & Publishing tools sidebars using 'latest code' and all works as expected.

Note once this is merged the sidebar will have to be versioned & republished.