Skip to content

Start extracting mandate and mandate editing from positions page#900

Merged
ManneG merged 17 commits intomainfrom
manage-positions
Nov 3, 2025
Merged

Start extracting mandate and mandate editing from positions page#900
ManneG merged 17 commits intomainfrom
manage-positions

Conversation

@ManneG
Copy link
Contributor

@ManneG ManneG commented Oct 6, 2025

Closes #571

@ManneG
Copy link
Contributor Author

ManneG commented Oct 6, 2025

What is left to do right now:

  • Redo the contents and styling of the mandate editing "tooltip"
  • Add a page at (app)/positions to get an overview, add, and mark positions as active/inactive
  • perhaps expose deleting positions but make it clear this is not something that should usually be done as old positions should still exist in the database. This would be if you mistakenly created a post. Maybe only be able to delete posts with no people in them
  • add calls to update groups in Authentik when posts are created

@ManneG
Copy link
Contributor Author

ManneG commented Oct 13, 2025

I have made some good progress

What is left to do right now:

  • Redo the contents and styling of the mandate editing "tooltip"
  • ponder if positions should be able to be created in the overview (Daniel said we have a syncing script that should be fixed, I thought this was the main thing with this issue but perhaps not)
  • review Anahita's changes Made design less ugly #898 to editing mandates as that will conflict with my changes.
  • there are some bugs, something not updating correctly in the database (thought I fixed this but I found the bug happening again)

@ManneG ManneG marked this pull request as ready for review November 3, 2025 19:04
@ManneG ManneG requested a review from Isak-Kallini November 3, 2025 19:07
@Isak-Kallini Isak-Kallini requested review from a team and RufusKoggRojder and removed request for a team and Isak-Kallini November 3, 2025 19:11
Copy link
Contributor

@RufusKoggRojder RufusKoggRojder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@ManneG ManneG merged commit de6f68d into main Nov 3, 2025
4 checks passed
@ManneG ManneG deleted the manage-positions branch November 3, 2025 20:52
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in CPU Nov 3, 2025
@ManneG ManneG mentioned this pull request Nov 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Add an admin interface for managing positions

2 participants