Added an URL to display a schedule #416

Merged

b24johka merged 1 commit from #415-add-schedule-URL into team_1_week_2

2026-04-20 06:56:53 +00:00

a24julot commented

2026-04-17 12:32:55 +00:00

Collaborator

Added an URL for the course to avoid error message.

Skärmbild 2026-04-17 143202.png

43 KiB

a24julot added 1 commit

2026-04-17 12:32:56 +00:00

Added an URL for the course to avoid error message. 90b49a6a70

a24julot added the

needs review

label

2026-04-17 12:38:12 +00:00

a24julot referenced this pull request

2026-04-17 12:51:56 +00:00

Add the course schedule URL to the schedule view #415

b24krila was assigned by b24johka

2026-04-17 13:48:33 +00:00

b24krila commented

2026-04-17 14:08:23 +00:00

Collaborator

Reviewed "Added an URL to display a schedule #416."

The code follows the conventions and rules set in the wiki.

The code ran without any problems and I could display the schedule view with out any errors.

The one point of question is the const dependency = null is there even a point of having it there if its null? As no dependecny is needed yet wouldn't it be easier to just remove it to be clearer?.

In conclusion the code does not have any real problems and can be merged.

Reviewed "Added an URL to display a schedule #416." The code follows the conventions and rules set in the wiki. The code ran without any problems and I could display the schedule view with out any errors. The one point of question is the `const dependency = null ` is there even a point of having it there if its null? As no dependecny is needed yet wouldn't it be easier to just remove it to be clearer?. In conclusion the code does not have any real problems and can be merged.

b24krila added

reviewed

and removed

needs review

labels

2026-04-17 14:08:43 +00:00