ddp/photoassistant
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

improve /view naming

Browse Source
This commit is contained in:
Damien De Paoli
2022-01-20 14:18:27 +11:00
parent 2dab42d712
commit 5461cb4036
2 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
TODO
View File

@@ -1,7 +1,7 @@
## GENERAL ## GENERAL
* for below.. make Options( request ) * for below.. make Options( request )
- know that a view is a "path", dont rely on orig_url - know that a view is a "path", dont rely on orig_url
- remove all "Options()" that are not set by a user-choice in the F/E and just make sure new "OPtions" sets all defaults as needed - remove all "Options()" that are not set by a user-choice in the F/E and just make sure new "Options" sets all defaults as needed
-- viewlist can work out new view_eids server side, and pass them back as json data -- viewlist can work out new view_eids server side, and pass them back as json data
- can consider an optim-- new_view page makes calls to viewlist to ADD json data only, so only trigger a new "viewlist" if we dont have data for that part of the eids - can consider an optim-- new_view page makes calls to viewlist to ADD json data only, so only trigger a new "viewlist" if we dont have data for that part of the eids

8
files.py
View File

@@ -549,8 +549,8 @@ def viewlist():
return render_template("viewer.html", current=current, eids=eids, objs=objs, OPT=OPT ) return render_template("viewer.html", current=current, eids=eids, objs=objs, OPT=OPT )
@login_required @login_required
@app.route("/new_view/<id>", methods=["GET"]) @app.route("/view/<id>", methods=["GET"])
def newview_img(id): def view(id):
OPT=Options( request ) OPT=Options( request )
objs = {} objs = {}
print( OPT ) print( OPT )
@@ -575,9 +575,9 @@ def newview_img(id):
################################################################################ ################################################################################
@app.route("/view/<id>", methods=["POST"]) @app.route("/view/<id>", methods=["POST"])
@login_required @login_required
def view_img(id): def view_img_post(id):
OPT=Options( request ) OPT=Options( request )
return redirect( "/new_view/" + id ); return redirect( "/view/" + id );
# route called from front/end - if multiple images are being transformed, each transorm == a separate call # route called from front/end - if multiple images are being transformed, each transorm == a separate call
# to this route (and therefore a separate transorm job. Each reponse allows the f/e to check the # to this route (and therefore a separate transorm job. Each reponse allows the f/e to check the