Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add JSON API #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add JSON API #13

wants to merge 1 commit into from

Conversation

astei
Copy link
Contributor

@astei astei commented Jun 30, 2020

Advantage over #11 is that it updates if more drama is added and it includes permalinks. In addition, this is going to be much faster as it runs on the same Cloudflare Worker, which as I understand is pretty hot :)

Unfortunately, this touches the site, but it's a preferable solution IMO.

@Chew
Copy link
Contributor

Chew commented Jun 30, 2020

updates if more drama is called

Not an advantage, mine does this too.

Only disadvantage is it would cost since each request is billed beyond 100,000 (assuming it would ever get this high).

Otherwise good, was trying to figure out permalinks since I'd have code it manually lol

@Chew
Copy link
Contributor

Chew commented Jun 30, 2020

Alright mine has permalink now.
Up to him now :)

edit oh god I sound like Bardy plugging my api instead lol

@mbax
Copy link

mbax commented Jul 1, 2020

I am willing to license your PR here as MIT.

@Chew
Copy link
Contributor

Chew commented Jul 2, 2020

These PRs are turning into regular drama...

@astei
Copy link
Contributor Author

astei commented Jul 2, 2020

Drama generator generates drama around the drama generator. Gosh are we getting meta...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants