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

Some new features added #77

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Some new features added #77

wants to merge 5 commits into from

Conversation

ziabary
Copy link

@ziabary ziabary commented Aug 15, 2016

I've added some methods to be able to automatically process arguments passed as URL queries which are essential in complex REST APIs

* URL queries can be passed to functions using $args keyword in function arguments. 
* You can define default value for arguments passed using URL queries by @param keyword in function documentation.
* You can now define a parent dir in order to server REST API in sub-paths
* temp directory has been defined as cache directory
Sample method which can receive URL queries added
@tyzoid
Copy link
Collaborator

tyzoid commented Jul 18, 2017

There are some interesting changes here, but I'll need to do some testing/refactoring to be able to merge this in. First of all, the tabs-vs-spaces indentation issue caused the entire RestServer file to conflict - something that will need to be redone in order to merge properly. Secondly, I have a feature that I had been working on that used this parameter as well, so I'll need to think about what to do on that one.

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

Successfully merging this pull request may close these issues.

2 participants