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

routes with parameters don't work #3

Closed
UseTheApi opened this issue Apr 28, 2017 · 0 comments
Closed

routes with parameters don't work #3

UseTheApi opened this issue Apr 28, 2017 · 0 comments

Comments

@UseTheApi
Copy link
Owner

UseTheApi commented Apr 28, 2017

url_discovery does not validate routes with params like:

@route("/test/app/<id>", methods=["GET"])
def test_adpp(id):
     return "HELLO"
UseTheApi added a commit that referenced this issue May 8, 2017
* matching endpoint to a view_function moved to validate_route function (module: utils) #4
* removed UrlDiscovery class #4
* multiple routes support #5
* tests without process #2
UseTheApi added a commit that referenced this issue May 8, 2017
method url_for requires a parameter to be passed (validating actual url_string)
otherwise throwing an exception, that why has_no_empty_params() existed before.

by using directly an rule.rule <string> (which indicates a route) all of it can be removed and routes with params can be aggregated from actual urls
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

No branches or pull requests

1 participant