feat: addition of a button below existing one for the routing to the …

[owens-hub-git]

Description

Strict frontend change in the home component where I added a button underneath the current view all for previous units. This still routes to the same component for previous units, but will be the start for the cross-unit dashboard development

Type of change

• New feature (non-breaking change which adds functionality)

How Has This Been Tested?

1. Run the project locally.
2. Opened the HomePage as a student
3. Changing layouting for responsiveness and routing
4. npm test locally

Testing Checklist:

• Tested in latest Chrome
• Tested in latest Safari
• Tested in latest Firefox

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have requested a review from @macite and @jakerenzella on the Pull Request

[cillynder]

The dashboard button should be above the history button as it would generally be more important.

[cillynder]

Suggested change

	aria-label="view all projects"
	aria-label="view previous projects"

[cillynder]

Suggested change

	<mat-icon style="color: white">history</mat-icon> View all
	<mat-icon style="color: white">history</mat-icon> View previous

Should be changed so it's distinct from the other button.

[cillynder]

Looks good to me!

[owens-hub-git]

@BrianDangDev
Could you please have a look at this PR to merge, Thankyou

[WaelAlahamdi]

Hi @owens-hub-git, I reviewed and tested the branch locally. It works well, but at the moment both buttons seem to go to the same page.
Based on the task, I expected the new button to open the dashboard. Could you please confirm whether this is the intended behaviour or if there is still another routing change to be added before approval?
Thanks.

[owens-hub-git]

@WaelAlahamdi, This was mentioned in the description about what should happen when clicking the button as we are still working on the page to route to

[WaelAlahamdi]

Thanks @owens-hub-git for clarifying. I reviewed and tested this locally, and it works as expected for the current scoped UI change. Approved based on the PR description.

[BrianDangDev]

Approved and Merged