Skip to content

feat: Department entity#9

Merged
jeppekroghitk merged 17 commits into
developfrom
feature/7792-department-entity
Jun 26, 2026
Merged

feat: Department entity#9
jeppekroghitk merged 17 commits into
developfrom
feature/7792-department-entity

Conversation

@jeppekroghitk

@jeppekroghitk jeppekroghitk commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Link to ticket

#7808

Description

Instead of having "Organisatorisk forankring" hardcoded as an ENUM, it has now been renamed to "Afdeling" with an acompanied Department entity.

Both Department and Contacts CRUDs have been moved to the administration panel, to declutter the main menu.

Screenshot of the result

N/A

Checklist

  • My code is covered by test cases.
  • My code passes our test (all our tests).
  • My code passes our static analysis suite.
  • My code passes our continuous integration process.

@jeppekroghitk jeppekroghitk changed the base branch from develop to feature/7792-general-creation-motivation June 25, 2026 13:03
@jeppekroghitk jeppekroghitk requested a review from tuj June 26, 2026 12:33

@tuj tuj left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved with minor thing that should be fixed

Comment thread src/Entity/Department.php Outdated

#[ORM\Entity(repositoryClass: DepartmentRepository::class)]
#[UniqueEntity(fields: ['name'], message: 'department.name_duplicate')]
class Department

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

extends AbstractEntity

Base automatically changed from feature/7792-general-creation-motivation to develop June 26, 2026 18:17
@jeppekroghitk jeppekroghitk merged commit 435b2eb into develop Jun 26, 2026
7 checks passed
@jeppekroghitk jeppekroghitk deleted the feature/7792-department-entity branch June 26, 2026 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants