Skip to content

Commit

Permalink
Merge pull request #416 from GermanZero-de/32-wording-handlungsfeld
Browse files Browse the repository at this point in the history
[#32] use the name Handlungsfeld instead of Sektor
  • Loading branch information
cafischer authored Apr 5, 2024
2 parents b26930b + 7fb5a96 commit 64f6865
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 13 deletions.
8 changes: 5 additions & 3 deletions cpmonitor/admin.py
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,9 @@ def edit_tasks(self, city: City):
This is the only entry point to reach the changelist for the tasks.
"""
list_url = _admin_url(Task, "changelist", city.id)
return format_html('<a href="{}">KAP bearbeiten</a>', list_url)
return format_html(
'<a href="{}">Handlungsfelder / Maßnahmen bearbeiten</a>', list_url
)

# ------ change / add page ------
save_on_top = True
Expand Down Expand Up @@ -315,7 +317,7 @@ def changelist_view(self, request):
if city_id and rules.is_allowed_to_edit(request.user, int(city_id)):
return super().changelist_view(request)

msg = "Bitte eine Stadt auswählen, für die Sektoren / Maßnahmen geändert werden sollen. Rechts davon 'KAP bearbeiten' wählen."
msg = "Bitte eine Stadt auswählen, für die Handlungsfelder / Maßnahmen geändert werden sollen. Rechts davon 'Handlungsfelder / Maßnahmen bearbeiten' wählen."
self.message_user(request, msg, messages.INFO)
return HttpResponseRedirect(_admin_url(City, "changelist", None))

Expand Down Expand Up @@ -386,7 +388,7 @@ def add_view(self, request, form_url="", extra_context=None):
if city_id and rules.is_allowed_to_edit(request.user, int(city_id)):
return super().add_view(request, form_url, extra_context)

msg = "Bitte eine Stadt auswählen, für die ein Sektor / eine Maßnahme hinzugefügt werden soll. Rechts davon 'KAP bearbeiten' wählen."
msg = "Bitte eine Stadt auswählen, für die Handlungsfelder / Maßnahmen geändert werden sollen. Rechts davon 'Handlungsfelder / Maßnahmen bearbeiten' wählen."
self.message_user(request, msg, messages.INFO)
return HttpResponseRedirect(_admin_url(City, "changelist", None))

Expand Down
2 changes: 1 addition & 1 deletion e2e_tests/database/test_database.json
Original file line number Diff line number Diff line change
Expand Up @@ -323,7 +323,7 @@
"frontpage": false,
"title": "Mobilität",
"teaser": "Auch in der Mobilität muss sich einiges ändern.",
"description": "Die Maßnahmen in diesem Sektor dienen der Darstellung der verschiedenen Möglichkeiten.",
"description": "Die Maßnahmen in diesem Handlungsfeld dienen der Darstellung der verschiedenen Möglichkeiten.",
"planned_start": null,
"planned_completion": null,
"responsible_organ": "",
Expand Down
16 changes: 8 additions & 8 deletions e2e_tests/test_admin.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ def test_should_succeed_when_logging_into_admin(live_server, page: Page):
def test_should_not_allow_duplicate_sectors(live_server, page: Page):
admin_login(live_server.url, page)

sector1 = "Sektor 1"
sector1 = "Handlungsfeld 1"
add_task(live_server.url, page, sector1)
add_task(live_server.url, page, sector1)

Expand All @@ -72,11 +72,11 @@ def test_should_not_allow_duplicate_sectors(live_server, page: Page):
def test_should_allow_add_when_same_title_in_another_sector(live_server, page: Page):
admin_login(live_server.url, page)

sector1 = "Sektor 1"
sector1 = "Handlungsfeld 1"
add_task(live_server.url, page, sector1)
add_task(live_server.url, page, "Personal Einstellen", sector1)

sector2 = "Sektor 2"
sector2 = "Handlungsfeld 2"
add_task(live_server.url, page, sector2)
add_task(live_server.url, page, "Personal Einstellen", sector2)

Expand All @@ -89,7 +89,7 @@ def test_should_allow_add_when_same_title_in_another_sector(live_server, page: P
def test_should_not_allow_add_when_same_title_in_same_sector(live_server, page: Page):
admin_login(live_server.url, page)

sector1 = "Sektor 1"
sector1 = "Handlungsfeld 1"
add_task(live_server.url, page, sector1)
add_task(live_server.url, page, "Personal Einstellen", sector1)
add_task(live_server.url, page, "personal einstellen", sector1)
Expand All @@ -105,8 +105,8 @@ def test_should_also_move_subtasks_when_dragging_a_task_with_subtasks_to_a_diffe
):
admin_login(live_server.url, page)

sector1 = "Sektor 1"
sector2 = "Sektor 2"
sector1 = "Handlungsfeld 1"
sector2 = "Handlungsfeld 2"
task = "To be dragged"
sub_task = "Subtask"
sub_sub_task = "Subsubtask"
Expand Down Expand Up @@ -146,8 +146,8 @@ def test_should_also_move_subtasks_when_dragging_a_task_with_subtasks_to_a_diffe
# ):
# admin_login(live_server.url, page)
#
# sector1 = "Sektor 1"
# sector2 = "Sektor 2"
# sector1 = "Handlungsfeld 1"
# sector2 = "Handlungsfeld 2"
# task1 = "Same Task Title"
# task2 = task1.lower()
#
Expand Down
2 changes: 1 addition & 1 deletion e2e_tests/test_task_page.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ def test_should_show_the_taskgroup_teaser(live_server, page: Page):
expect(
page.locator(
"p",
has_text="Die Maßnahmen in diesem Sektor dienen der Darstellung der verschiedenen Möglichkeiten.",
has_text="Die Maßnahmen in diesem Handlungsfeld dienen der Darstellung der verschiedenen Möglichkeiten.",
)
).to_be_visible()

Expand Down

0 comments on commit 64f6865

Please sign in to comment.