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

Discuss how the API should be advertised #3

Open
1 of 2 tasks
gdude2002 opened this issue Aug 13, 2014 · 14 comments
Open
1 of 2 tasks

Discuss how the API should be advertised #3

gdude2002 opened this issue Aug 13, 2014 · 14 comments

Comments

@gdude2002
Copy link
Contributor

  • Is it a public API?
  • If so, how do we document it?
    • I'd suggest an index page that details how the API should be used.
@Ribesg
Copy link
Member

Ribesg commented Aug 13, 2014

Well, for any documentation, I suggest using Markdown, it's simpler.

It is a public API imo. For example, IRC bots will love to use it.

@gdude2002
Copy link
Contributor Author

I'd have thought it was a public API as well. And, yep, I can do Markdown, it's really fairly simple to do with Python. I think it should definitely be an index page rather than on GitHub, though.

@Ribesg
Copy link
Member

Ribesg commented Aug 13, 2014

Yeah didn't mean it should be a Github wiki.

@gdude2002 gdude2002 self-assigned this Aug 13, 2014
@gdude2002
Copy link
Contributor Author

Here's how it looks right now.

@gdude2002
Copy link
Contributor Author

What do we think of this?

@greatman
Copy link
Member

Fine to me.

@robbie-wilson
Copy link
Contributor

Is it possible to have /format/ not be case sensitive?

@gdude2002
Copy link
Contributor Author

Is it possible to have /format/ not be case sensitive?

Huh? Case insensitive routing?

When would you need to use uppercase or mixed case instead of lowercase?

Yes, it's possible, technically speaking, but it's a pain in the ass.

@robbie-wilson
Copy link
Contributor

If its a pain in the ass it doesn't matter, just a suggestion.

@gdude2002
Copy link
Contributor Author

Well, I'm not saying I wouldn't do it regardless, I'm just trying to think of a use case for it :P

@Ribesg
Copy link
Member

Ribesg commented Aug 14, 2014

It's not hard to do ._.

@gdude2002
Copy link
Contributor Author

Is it possible to have /format/ not be case sensitive?

9845409...55b12bf

@lol768
Copy link

lol768 commented Aug 14, 2014

Use the Accept headers instead of the URL to specify formats.

@gdude2002
Copy link
Contributor Author

Dumb clients won't do that (and noobs won't know how to do that); but I could add it as an optional way of doing things.

gdude2002 added a commit that referenced this issue Aug 18, 2014
… routes. Note that /api/ROUTE won't be registered unless you use the hybrid_route function.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants