[pyramid] switch routing to be fully pyramid only
We need to start moving the code back to pyramid. We should start with making routes work only in PYRAMID, if that's possible.
- need to investigate if we can make it work
- we should have a one common place to define those like we had before
- can we support pylons DELETE/PUT fake request mapping, or should we re-do all the routes not to use them (it's really a POST faked by special parameter to mimic DELETE/PUT)
#1 Updated by Daniel D over 3 years ago
One issue with moving urls to pyramid is that generating urls for tests can't use the url generator.
You can't use
request.route_path('someroute') unless there is a request context - which means the tests would have to be done as:
url = ADMIN_PREFIX + '/some/route/' + str(id) vs url = request.route_path('someroute', id=id)
Although this might be good in actually finding route changing mistakes.
#8 Updated by Redmine Integration over 2 years ago
pullrequest created by marcink (status: under_review). https://internal-code.rhodecode.com/rhodecode-enterprise-ce/pull-request/2892
#9 Updated by Redmine Integration over 2 years ago
Commit c94a1c499e73 by Marcin Kuzminski email@example.com on default branch references this issue. https://internal-code.rhodecode.com/rhodecode-enterprise-ce/changeset/c94a1c499e73ad6b74bab6fde2ad274858d48ded
#10 Updated by Redmine Integration over 2 years ago
pullrequest merged by marcink (status: approved). https://internal-code.rhodecode.com/rhodecode-enterprise-ce/pull-request/2892