Skip to content

Feature/43 implement department read functionality#79

Open
DrDeathDrop wants to merge 21 commits into
mainfrom
feature/43-Implement-Department-Read-Functionality
Open

Feature/43 implement department read functionality#79
DrDeathDrop wants to merge 21 commits into
mainfrom
feature/43-Implement-Department-Read-Functionality

Conversation

@DrDeathDrop

Copy link
Copy Markdown
Contributor

No description provided.

@DrDeathDrop
DrDeathDrop requested a review from a team as a code owner June 23, 2026 10:51
@DrDeathDrop

Copy link
Copy Markdown
Contributor Author

A bug persists where the update won't work, if we try to update the faculty of the department. The issue is in the backend and is fixed, it's a different PR.

@DjesikaV DjesikaV left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We agreed to use inject() instead of constructor injection.

Ivcho added 4 commits June 25, 2026 17:44
# Conflicts:
#	uniplanWeb/src/app/core/shared/main-panel/main-panel.html
#	uniplanWeb/src/app/core/shared/main-panel/main-panel.ts
#	uniplanWeb/src/app/core/shared/navmenu-component/navmenu-component.html
#	uniplanWeb/src/app/core/shared/navmenu-component/navmenu-component.ts
@DrDeathDrop
DrDeathDrop force-pushed the feature/43-Implement-Department-Read-Functionality branch from 4fdcced to a42ebbe Compare June 25, 2026 15:08
Comment thread uniplanWeb/src/app/features/department/department-add-form/department-add-form.ts Outdated
Comment thread uniplanWeb/src/app/features/department/department-filters/department-filters.ts Outdated
Comment thread uniplanWeb/src/app/features/department/department-filters/department-filters.ts Outdated

@PIPetkova19 PIPetkova19 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change detection problem; Use signals

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.

3 participants