search

lenscas / rp_tracker

A webbased roleplay manager API
MIT License
1 stars 1 forks source link

change to restful api #39

Closed lenscas closed 7 years ago

lenscas commented 8 years ago

Using https://github.com/chriskacerguis/codeigniter-restserver it should be possible to change the API to a restful api.

This goes nicely with #37 as that will break compatibility anyway and the urls that are used for the api will need to change anyway.

edit: put the ajax to api part in issue #41

lenscas commented 7 years ago

Actually, after taking a better look at codeigniters routing system it is now possible to use the request method in the routes.

I like this method more as it means I don't have to add more third_party files and I didn't like how with that class you was restricted to a single function name with some stuff at the end of the name to filter by request method (thus for example users_post()). Where as with going with build in route system the function names can still be whatever (thus a nice register() ).

It does mean that we loose all the different kinds of ways the class adds to send data but considering that of all those XML was the only one that might looked useful that is probably not a big loss.

lenscas commented 7 years ago

after 0f8f45a the url's should be stable.