From 29645fd07086d0b10c8a645d204487f1d2b6b03e Mon Sep 17 00:00:00 2001 From: Bryan Newbold Date: Wed, 3 Apr 2019 18:41:42 -0700 Subject: basic annotation view/create --- python/fatcat_web/routes.py | 44 ++++++++++++++++--- python/fatcat_web/templates/editgroup_view.html | 58 ++++++++++++++++++++++--- 2 files changed, 90 insertions(+), 12 deletions(-) (limited to 'python/fatcat_web') diff --git a/python/fatcat_web/routes.py b/python/fatcat_web/routes.py index 7a148e94..e20cd3c5 100644 --- a/python/fatcat_web/routes.py +++ b/python/fatcat_web/routes.py @@ -6,7 +6,7 @@ from flask import Flask, render_template, send_from_directory, request, \ from flask_login import login_required from flask_wtf.csrf import CSRFError -from fatcat_client import Editgroup +from fatcat_client import Editgroup, EditgroupAnnotation from fatcat_client.rest import ApiException from fatcat_tools.transforms import * from fatcat_web import app, api, auth_api, priv_api @@ -313,19 +313,51 @@ def editgroup_view(ident): try: eg = api.get_editgroup(str(ident)) eg.editor = api.get_editor(eg.editor_id) + eg.annotations = api.get_editgroup_annotations(eg.editgroup_id, expand="editors") except ApiException as ae: abort(ae.status) # TODO: idomatic check for login? - auth_to_submit = False - auth_to_accept = False + auth_to = dict( + submit=False, + accept=False, + annotate=False, + ) if session.get('editor'): user = load_user(session['editor']['editor_id']) + auth_to['annotate'] = True if user.is_admin or user.editor_id == eg.editor_id: - auth_to_submit = True + auth_to['submit'] = True if user.is_admin: - auth_to_accept = True + auth_to['accept'] = True return render_template('editgroup_view.html', editgroup=eg, - auth_to_submit=auth_to_submit, auth_to_accept=auth_to_accept) + auth_to=auth_to) + +@app.route('/editgroup//annotation', methods=['POST']) +@login_required +def editgroup_create_annotation(ident): + app.csrf.protect() + comment_markdown = request.form.get('comment_markdown') + if not comment_markdown: + app.logger.info("empty comment field") + abort(400) + # on behalf of user... + user_api = auth_api(session['api_token']) + try: + eg = user_api.get_editgroup(str(ident)) + if eg.changelog_index: + flash("Editgroup already accepted") + abort(400) + ega = EditgroupAnnotation( + comment_markdown=comment_markdown, + extra=None, + ) + user_api.create_editgroup_annotation(eg.editgroup_id, ega) + except ApiException as ae: + app.logger.info(ae) + abort(ae.status) + return redirect('/editgroup/{}'.format(ident)) + +# XXX: editor's annotations @app.route('/editgroup//accept', methods=['POST']) @login_required diff --git a/python/fatcat_web/templates/editgroup_view.html b/python/fatcat_web/templates/editgroup_view.html index 37771273..2f61671d 100644 --- a/python/fatcat_web/templates/editgroup_view.html +++ b/python/fatcat_web/templates/editgroup_view.html @@ -1,4 +1,6 @@ {% extends "base.html" %} +{% import "entity_macros.html" as entity_macros %} + {% block body %} {% macro edit_list(edits, entity_type, entity_name) -%} @@ -37,13 +39,13 @@ {% block editgroupheader %} {% if not editgroup.changelog_index %}
- {% if auth_to_accept %} + {% if auth_to.accept %}

{% endif %} - {% if auth_to_submit %} + {% if auth_to.submit %} {% if editgroup.submitted %}
@@ -67,6 +69,11 @@
editgroup {{ editgroup.editgroup_id }}
{% endblock %} +

What is an editgroup? +An editgroup is a set of entity edits, bundled together into a coherent, +reviewable bundle. +
+
Status: {% if editgroup.changelog_index %} Merged (Changelog #{{ editgroup.changelog_index }}) @@ -95,11 +102,50 @@ {{ edit_list(editgroup.edits.webcaptures, "webcapture", "Web Capture") }}

-
-

What is an editgroup? -An editgroup is a set of entity edits, bundled together into a coherent, -reviewable bundle. +

Comments and Annotations

+{% for annotation in editgroup.annotations|reverse %} +
+
+ {% if annotation.editor.is_bot %} + + {% else %} + + {% endif %} + {{ annotation.editor.username}} + {% if annotation.editor.is_admin %} + Admin + {% endif %} + at {{ annotation.created.strftime("%Y-%m-%d %H:%M:%S") }} +
+ {% if annotation.extra %} +
+ {{ entity_macros.extra_metadata(annotation.extra) }} +
+ {% endif %} +
+ {{ annotation.comment_markdown }} +
+
+{% else %} + None! +{% endfor %} + +{% if auth_to.annotate and not editgroup.changelog_index %} +
+

Add Comment

+ + +
+ +
+ +
+
+
+{% endif %} {% endblock %} -- cgit v1.2.3