#112 Research/test schedule card

a22erigr commented

2026-04-02 12:25:38 +00:00

Collaborator

added api to fetch data from timeedit. Added new view for the schedule

a22erigr added 1 commit

2026-04-02 12:25:39 +00:00

#112 Research/test schedule card e7342a3778

added api to fetch data from timeedit. Added new view for the schedule

a22erigr added the

needs review

label

2026-04-02 12:26:07 +00:00

c24elipe commented

2026-04-02 12:51:33 +00:00

Collaborator

Related issue is "[Card] Research/test schedule card #112"

b24idalu was assigned by c24elipe

2026-04-02 12:52:34 +00:00

b24idalu commented

2026-04-02 13:31:47 +00:00

Collaborator

Comments do not follow set code standard, for example in useTimeedit it says "timestamp, temp" and in ScheduleView a single line comment is on 2 rows (Idk how correct that needs to be, could skip that bit and just change the first temp comment). Should only the next date be shown? I think more information/context is needed, such as which room is being shown or between what times. Looks good and works well otherwise!

a22erigr added 1 commit

2026-04-03 09:36:58 +00:00

#112 Fixed comments 248fa2800a

The feedback from the first pull request has been fixed (Comments not
following standard)

a22erigr changed target branch from main to team_2_week_1

2026-04-07 07:39:43 +00:00

b24johka commented

2026-04-07 08:33:52 +00:00

Collaborator

Tested the branch locally and it worked very well. Has informed the developer that we need unit testing before initialisation of merge commmit.

a22erigr was assigned by c24elipe

2026-04-07 13:23:40 +00:00

a22erigr commented

2026-04-07 13:24:16 +00:00

Author

Collaborator

Working on it

a22erigr commented

2026-04-08 16:35:03 +00:00

Author

Collaborator

Because Breakout of duplicate code in frontend/src/features #159 refactors the function that needed unit test in to one shared function only unit test for the shared function was deemed necessary. A new branch was created from this one. The refactoring implemented and adhering unit tests

Because [ Breakout of duplicate code in frontend/src/features #159 ](https://git.webug.se/Andras/BoundlessFlowCampus2K/issues/159) refactors the function that needed unit test in to one shared function only unit test for the shared function was deemed necessary. A new branch was created from this one. The refactoring implemented and adhering unit tests

a22erigr commented

2026-04-08 16:37:35 +00:00

Author

Collaborator

Important: This branch should only be merged into team_2_week_1 after following merge is complete #159 Refactored code to use DRY and added test #163

**Important:** This branch should only be merged into team_2_week_1 after following merge is complete [ #159 Refactored code to use DRY and added test #163 ](https://git.webug.se/Andras/BoundlessFlowCampus2K/pulls/163)

c24elipe added 6 commits

2026-04-09 07:37:32 +00:00

#159 Refactored code to use DRY and added test f40e786675

Renamed test file and added possible test cc9fc9b006

implemented jest test b9ff27f168

testing the shared apifetch function

fixes after review 68b4f74131

- Name change: SharedFetures.js ->SharedFeatures.js
 - Change import in effected file
- Implemented unit test for failure case
 - Error 500
- Removed debug code

Quick name fix before merge b05fbd9305

I just capitalized "UseApi" and the description to comply with the coding standard.

Merge pull request '#159 Refactored code to use DRY and added test' (#163 ) from #159-Breakout-of-duplicate-code-in-frontend/src/features into #112-Card]-Research/test-schedule-card 46838af161

Reviewed-on: #163

c24elipe added 1 commit

2026-04-09 07:39:20 +00:00

Merge branch 'team_2_week_1' into #112-Card]-Research/test-schedule-card 98fb231aa5

c24elipe merged commit e8cff73e06 into team_2_week_1

2026-04-09 07:40:14 +00:00

c24elipe deleted branch #112-Card]-Research/test-schedule-card

2026-04-09 07:40:15 +00:00

c24elipe referenced this pull request from a commit

2026-04-09 07:40:17 +00:00

Merge pull request '#112 Research/test schedule card' (#137) from #112-Card]-Research/test-schedule-card into team_2_week_1

a24noabe removed the

needs review

label

2026-04-14 13:04:57 +00:00

b25marem referenced this pull request from a commit

2026-05-15 08:55:59 +00:00

Merge pull request '#112 Research/test schedule card' (#137) from #112-Card]-Research/test-schedule-card into team_2_week_1

Rows
Columns

#112 Research/test schedule card #137