Skip to content

Conversation

amirhossein-haerian
Copy link
Contributor

@amirhossein-haerian amirhossein-haerian commented Mar 18, 2025

This PR should not be merged.

This is a PR to test whether we can show the generated data for Kurslista in Ladok syllabuses.
The data is being converted to HTML inside studadm-om-kursen-packages. another PR is being created on that Repository.

This PR is only working for TITEH program only now.

here is an example: http://localhost:3000/student/kurser/program/TITEH/20181/arskurs3?l=en

Screen shots of the result:

screencapture-localhost-3000-student-kurser-program-TITEH-20181-arskurs1-2025-03-18-14_03_12
screencapture-localhost-3000-student-kurser-program-TITEH-20181-arskurs2-2025-03-18-14_03_38
screencapture-localhost-3000-student-kurser-program-TITEH-20181-arskurs3-2025-03-18-14_04_07

amirhossein-haerian and others added 4 commits October 17, 2024 09:31
* feat(KP-367): the search is now updated with the new data comming from ladok

* feat(KP-367): search filters are now working as expected in phase 1

* feat(KP-367): formatted course credits has been used

* fix(KP-367): one comment is added

* fix(KP-367): change condition for triggering noQueryProvidedDispatch

* fix(KP-367): add errorUnKnown as default error message for alerts

* the front-end is updated after the upgrade in grouping results from ladok

* feat(KP-367): search parameters chnged to only semesters

* feat(KP-367): search parameters are changed and the task is ready

* change the ladok API usage based on changes in ladok mellan lager
* feat(KP-373): replace the old search with the newer version

* fix(KP-373): remove old search from more places
@amirhossein-haerian amirhossein-haerian changed the base branch from master to upl-main March 18, 2025 13:38
Copy link
Contributor

@allazis allazis left a comment

Choose a reason for hiding this comment

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

Really good job on this! The example looks great ⭐

Looking forward to see further work on this, especially for the hybrid program syllabuses using both this and the structures.

@allazis allazis marked this pull request as draft April 1, 2025 11:57
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