Too many requests when selecting course from the sidebar

[Malorn44]

Describe the bug
When selecting a course in the sidebar, too many requests are made and there is a very noticeable slowdown. It also looks like there are more requests, the more sections a course has.

To Reproduce

1. Select a couple of courses
2. Open the sidebar
3. Unselect and reselect a course (if you have your network tab open in your developer tools, you'll see there are a lot more requests than there should be)

Expected behavior
Ideally, this should only make a single request

[LuminaSapphira]

This is a result of the current implementation of SelectionService calling addListing on SidebarService every time a section is added, which happens as part of a loop when toggleCourse is called in SelectionService. This should be easily hotfixable, though the refactored selection service will fix this issue as well.

See code:

yacs/web/src/app/services/selection.service.ts

Lines 77 to 79 in 1376fb2

	course.sections.forEach((s) => {
	this.addSection(s);
	});

yacs/web/src/app/services/selection.service.ts

Line 54 in 1376fb2

this.sidebarService.addListing(section.listing);

[Malorn44]

alright great 👍
I'll keep this issue open of course until it is confirmed to be fixed. Glad this is being worked on.