static-dev / spike-datocms

Spike plugin for easily consuming datocms data
Other
15 stars 8 forks source link

Changes to API #4

Open stefanoverna opened 7 years ago

stefanoverna commented 7 years ago

And here's an example of usage: https://github.com/stefanoverna/spike-datocms-example/blob/master/app.js

The main changes:

stefanoverna commented 7 years ago

@jescalan if you like the overall direction of the proposed changes, I can finalize things and write tests. Let me know if I can start :)

jescalan commented 7 years ago

@stefanoverna Yeah, I would say don't worry about the meta tags helper, I can take that section, but as far as the primary piece with using the client, I think this is good to start testing!

Interesting with the websockets though, this is really cool. I can also hook this up to the watcher, don't worry about that part.

stefanoverna commented 7 years ago

Hi @jescalan, we're receiving requests regarding this PR on our support chat.. looking forward to close it! 😃 I'm not very familiar with the internals of spike so I would really need some help of yours :)

screenshot 2017-08-29 12 30 27

jescalan commented 7 years ago

So, this is massively breaking and still has no tests at all, so I can't really merge it yet, it still needs a bunch of work. Also, what's being asked there can already be accomplished with the plugin as it exists - you're welcome to direct them here to the issues section in this repo with questions!