Skip to content
This repository has been archived by the owner on Sep 7, 2020. It is now read-only.

New data/collection API #889

Closed
MoOx opened this issue Nov 22, 2016 · 2 comments · Fixed by #936
Closed

New data/collection API #889

MoOx opened this issue Nov 22, 2016 · 2 comments · Fixed by #936

Comments

@MoOx
Copy link
Owner

MoOx commented Nov 22, 2016

Currently our api is mostly a legacy from the initial version and is not really simple and pretty even if "it works". We have some limitations and some stuff are not optimal

I also want to add this

Currently we could use more tricks but we discussed already about smarter ideas on how to have a better intuitive, minimalist API which will even unlock feature like pagination #657

⚠️ I didn't mention graphql since I don't think it will be really intuitive for most people. Gastby 1.0 branch is probably doing awesome stuff with that, but I have to admit I found that a bit overkill, especially for simple websites or documentation. It's probably very powerful but maybe a bit too much for most cases when you have data in collection(s) and just want to iterate on it (blog, photo gallery etc). You tell me.


Fresh API ideas have been exposed here #713 (comment)

I will start to work on something based on what @bloodyowl developed in this thread but feel free to comment and dissuss here about the approach YOU think is the best. There is no bad ideas, just ideas we can learn from ;)

@MoOx
Copy link
Owner Author

MoOx commented Dec 8, 2016

I am thinking about rewriting Phenomic using Next.js.

Pros:

Cons:

Next will offer an official API to handle static rendering vercel/next.js#70


Anyway, we might probably stop relying on a loader to consume markdown|* files with probably a simple interface like metalsmith that will still allow a plugin API.
Why? Because we need something generic to allow consuming files or promise (rest api, graphql api...)

@bloodyowl
Copy link
Contributor

Going to be fixed with #925

@ryandeussing ryandeussing mentioned this issue Dec 26, 2016
@MoOx MoOx mentioned this issue Jan 12, 2017
Merged
@MoOx MoOx closed this as completed in #936 May 11, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants