Fs 4633 merge configure view (#39)

* fs-4606 basic create section * Can clone template and remove form from section * Updated clone form to fix default next page ids * fix numbering of new sections and forms * remove section from round * allow importing multiple forms with same page paths * fs-4606 basic create section * Can clone template and remove form from section * Updated clone form to fix default next page ids * fix numbering of new sections and forms * remove section from round * allow importing multiple forms with same page paths * fs-4606 basic create section * Can clone template and remove form from section * fix numbering of new sections and forms * remove section from round * fix rebase errors * WIP merging frontends * Tidied up journey * fs-4606 basic create section * Can clone template and remove form from section * Updated clone form to fix default next page ids * fix numbering of new sections and forms * remove section from round * allow importing multiple forms with same page paths * fs-4606 basic create section * Can clone template and remove form from section * fix numbering of new sections and forms * remove section from round * fs-4606 basic create section * Can clone template and remove form from section * Updated clone form to fix default next page ids * fix numbering of new sections and forms * remove section from round * fix rebase errors * WIP merging frontends * Tidied up journey * add date format after merge * Fix migration post merge * Tidy up - breadcrumbs, fix remove section, add info to headings
communitiesuk · Sep 18, 2024 · f0a24f0 · f0a24f0
1 parent 2703a86
commit f0a24f0
Show file tree

Hide file tree

Showing 20 changed files with 410 additions and 389 deletions.
diff --git a/app/all_questions/metadata_utils.py b/app/all_questions/metadata_utils.py
@@ -357,6 +357,7 @@ def build_components_from_page(
         title, text = determine_title_and_text_for_component(
             c, include_html_components=include_html_components, form_lists=form_lists
         )
+        field_type = c["type"]
         if not title and not text:
             continue
 
@@ -386,6 +387,7 @@ def build_components_from_page(
             "title": title,
             "text": text,
             "hide_title": c["options"]["hideTitle"] if "hideTitle" in c["options"] else False,
+            "type": field_type,
         }
         components.append(component)
     return components

diff --git a/app/blueprints/fund_builder/routes.py b/app/blueprints/fund_builder/routes.py
@@ -33,7 +33,6 @@
 from app.db.queries.fund import get_all_funds
 from app.db.queries.fund import get_fund_by_id
 from app.db.queries.round import add_round
-from app.db.queries.round import get_all_rounds
 from app.db.queries.round import get_round_by_id
 from app.export_config.generate_all_questions import print_html
 from app.export_config.generate_form import build_form_json
@@ -56,37 +55,26 @@
 )
 
 
-@build_fund_bp.route("/fund/round/configure", methods=["GET", "POST"])
-def configure_round():
-    """
-    Renders a template providing a drop down list of funds. If a fund is selected, renders its config info
-    """
-    all_rounds = get_all_rounds()
-
-    params = {
-        "all_rounds": [{"text": f"{r.short_name} - {r.title_json['en']}", "value": str(r.round_id)} for r in all_rounds]
-    }
-    if round_id := request.form.get("round_id") or request.args.get("round_id"):
-        round = get_round_by_id(round_id)
-        params["round"] = round
-        params["selected_round_id"] = round_id
-
-    return render_template("application.html", **params)
+@build_fund_bp.route("/")
+def index():
+    return render_template("index.html")
 
 
 @build_fund_bp.route("/fund/round/<round_id>/section", methods=["GET", "POST"])
 def section(round_id):
+    round_obj = get_round_by_id(round_id)
+    fund_obj = get_fund_by_id(round_obj.fund_id)
     form: SectionForm = SectionForm()
     form.round_id.data = round_id
     params = {
         "round_id": str(round_id),
     }
+    existing_section = None
     if request.args.get("action") == "remove":
         update_section(request.args.get("section_id"), {"round_id": None})  # TODO remove properly
-        return redirect(url_for("build_fund_bp.configure_round", round_id=round_id, _method="POST", code=307))
+        return redirect(url_for("build_fund_bp.build_application", round_id=round_id))
     if form.validate_on_submit():
-        round = get_round_by_id(form.round_id.data)
-        count_existing_sections = len(round.sections)
+        count_existing_sections = len(round_obj.sections)
         if form.section_id.data:
             update_section(
                 form.section_id.data,
@@ -103,8 +91,8 @@ def section(round_id):
                 }
             )
 
-        flash(f"Saved section {form.name_in_apply_en.data}")
-        return redirect(url_for("build_fund_bp.configure_round"), code=307)
+        # flash(f"Saved section {form.name_in_apply_en.data}")
+        return redirect(url_for("build_fund_bp.build_application", round_id=round_obj.round_id))
     if section_id := request.args.get("section_id"):
         existing_section = get_section_by_id(section_id)
         form.section_id.data = section_id
@@ -115,6 +103,14 @@ def section(round_id):
             for f in get_all_template_forms()
         ]
 
+    params["breadcrumb_items"] = [
+        {"text": fund_obj.name_json["en"], "href": url_for("build_fund_bp.view_fund", fund_id=fund_obj.fund_id)},
+        {
+            "text": round_obj.title_json["en"],
+            "href": url_for("build_fund_bp.build_application", fund_id=fund_obj.fund_id, round_id=round_obj.round_id),
+        },
+        {"text": existing_section.name_in_apply_json["en"] if existing_section else "Add Section", "href": "#"},
+    ]
     return render_template("section.html", form=form, **params)
 
 
@@ -148,23 +144,35 @@ def view_fund():
     Renders a template providing a drop down list of funds. If a fund is selected, renders its config info
     """
     params = {"all_funds": all_funds_as_govuk_select_items(get_all_funds())}
+    fund = None
     if request.method == "POST":
         fund_id = request.form.get("fund_id")
+    else:
+        fund_id = request.args.get("fund_id")
+    if fund_id:
         fund = get_fund_by_id(fund_id)
         params["fund"] = fund
         params["selected_fund_id"] = fund_id
+    params["breadcrumb_items"] = [
+        {"text": "Home", "href": url_for("build_fund_bp.index")},
+        {"text": fund.title_json["en"] if fund else "Manage Application Configuration", "href": "#"},
+    ]
 
-    return render_template("view_fund_config.html", **params)
+    return render_template("fund_config.html", **params)
 
 
 @build_fund_bp.route("/fund/round/<round_id>/application_config")
-def view_app_config(round_id):
+def build_application(round_id):
     """
     Renders a template displaying application configuration info for the chosen round
     """
     round = get_round_by_id(round_id)
     fund = get_fund_by_id(round.fund_id)
-    return render_template("view_application_config.html", round=round, fund=fund)
+    breadcrumb_items = [
+        {"text": fund.name_json["en"], "href": url_for("build_fund_bp.view_fund", fund_id=fund.fund_id)},
+        {"text": round.title_json["en"], "href": "#"},
+    ]
+    return render_template("build_application.html", round=round, fund=fund, breadcrumb_items=breadcrumb_items)
 
 
 @build_fund_bp.route("/fund/<fund_id>/round/<round_id>/clone")
@@ -176,16 +184,6 @@ def clone_round(round_id, fund_id):
     return redirect(url_for("build_fund_bp.view_fund", fund_id=fund_id))
 
 
-@build_fund_bp.route("/fund/round/<round_id>/assessment_config")
-def view_assess_config(round_id):
-    """
-    Renders a template displaying assessment configuration info for the chosen round
-    """
-    round = get_round_by_id(round_id)
-    fund = get_fund_by_id(round.fund_id)
-    return render_template("view_assessment_config.html", round=round, fund=fund)
-
-
 @build_fund_bp.route("/fund", methods=["GET", "POST"])
 def fund():
     """
@@ -298,7 +296,39 @@ def view_all_questions(round_id):
         lang="en",
     )
     html = print_html(print_data)
-    return render_template("view_questions.html", round=round, fund=fund, question_html=html)
+    return render_template(
+        "view_questions.html",
+        round=round,
+        fund=fund,
+        question_html=html,
+        title=f"All Questions for {fund.short_name} - {round.short_name}",
+    )
+
+
+@build_fund_bp.route("/fund/round/<round_id>/all_questions/<form_id>", methods=["GET"])
+def view_form_questions(round_id, form_id):
+    """
+    Generates the form data for this form, then uses that to generate the 'All Questions'
+    data for that form and returns that to render in a template.
+    """
+    round = get_round_by_id(round_id)
+    fund = get_fund_by_id(round.fund_id)
+    form = get_form_by_id(form_id=form_id)
+    section_data = [
+        {
+            "section_title": f"Preview of form [{form.name_in_apply_json['en']}]",
+            "forms": [{"name": form.runner_publish_name, "form_data": build_form_json(form)}],
+        }
+    ]
+
+    print_data = generate_print_data_for_sections(
+        section_data,
+        lang="en",
+    )
+    html = print_html(print_data, True)
+    return render_template(
+        "view_questions.html", round=round, fund=fund, question_html=html, title=form.name_in_apply_json["en"]
+    )
 
 
 @build_fund_bp.route("/create_export_files/<round_id>", methods=["GET"])

diff --git a/app/blueprints/fund_builder/templates/application.html b/app/blueprints/fund_builder/templates/application.html
diff --git a/app/blueprints/fund_builder/templates/build_application.html b/app/blueprints/fund_builder/templates/build_application.html
@@ -0,0 +1,60 @@
+{% extends "base.html" %}
+{%- from "govuk_frontend_jinja/components/button/macro.html" import govukButton -%}
+{%- from "govuk_frontend_jinja/components/accordion/macro.html" import govukAccordion -%}
+{%- from "govuk_frontend_jinja/components/summary-list/macro.html" import govukSummaryList -%}
+{%- from "govuk_frontend_jinja/components/select/macro.html" import govukSelect -%}
+{% block content %}
+    <div class="govuk-grid-row">
+        <div class="govuk-grid-column-full">
+            <h1 class="govuk-heading-l">Build Application</h1>
+            <h2 class="govuk-heading-m">{{ fund.short_name }} - {{round.title_json["en"]}} ({{ round.short_name }})</h2>
+        </div>
+    </div>
+    {{ govukButton({
+        "text": "View All Questions",
+        "href": url_for("build_fund_bp.view_all_questions", round_id=round.round_id) ,
+    "classes": "govuk-button--secondary"
+    })
+    }}
+    {{ govukButton({
+        "text": "Create export files for round",
+        "href": url_for("build_fund_bp.create_export_files", round_id=round.round_id) ,
+    "classes": "govuk-button--secondary"
+    })
+    }}
+    <ul class="app-task-list__items">
+        {{ govukButton({
+                "text": "Add Section",
+                "href": url_for("build_fund_bp.section", round_id=round.round_id) ,
+        "classes": "govuk-button--secondary"
+        })
+        }}
+        {% for section in round.sections %}
+        <li class="">
+                <span class="app-task-list__task-name">
+                    <h3 class="govuk-heading-m">{{ section.index }}. {{ section.name_in_apply_json["en"] }}</h3>
+                </span>
+                <span class="app-task-list__task-actions">
+                    <a href='{{ url_for("build_fund_bp.section", round_id=round.round_id, section_id=section.section_id) }}'>Edit</a>
+                    <a href='{{ url_for("build_fund_bp.section",round_id=round.round_id, section_id=section.section_id, action="remove") }}'>Remove</a>
+                </span>
+                <ul class="app-task-list__items">
+                    {% for form in section.forms %}
+                        <li class="app-task-list__item task-list__new-design">
+                            <span class="app-task-list__task-name">
+                                <h3 class="govuk-body">
+                                    {{ section.index }}.&nbsp;{{ form.section_index }}.&nbsp;{{ form.name_in_apply_json["en"] }}
+                                </h3>
+                            </span>
+                            <span class="app-task-list__task-actions">
+                                <a href="{{ url_for('build_fund_bp.view_form_questions', round_id=round.round_id, form_id=form.form_id) }}">View</a>&nbsp;
+                                <a href='{{ url_for("build_fund_bp.download_form_json", form_id=form.form_id) }}'>Download</a>&nbsp;
+                                <a href='{{ url_for("build_fund_bp.preview_form", form_id=form.form_id) }}'>Preview</a>&nbsp;
+                            </span>
+                        </li>
+                    {% endfor %}
+                </ul>
+            </li>
+        {% endfor %}
+    </ul>
+{% endblock content %}