Skip to content
This repository has been archived by the owner on Sep 27, 2024. It is now read-only.

Project cards #65

Merged
merged 5 commits into from
Mar 11, 2024
Merged

Project cards #65

merged 5 commits into from
Mar 11, 2024

Conversation

EmmaVandewalle
Copy link
Contributor

In frontend/src/pages/student/HomeStudent.tsx heb ik enkele statische objecten gezet van , zodat de structuur getoond wordt met oneven aantal kaarten en dat het een scroll veroorzaakt.

@EmmaVandewalle EmmaVandewalle added this to the Milestone 1 milestone Mar 9, 2024
@EmmaVandewalle EmmaVandewalle requested a review from matt01y March 9, 2024 15:47
@EmmaVandewalle EmmaVandewalle self-assigned this Mar 9, 2024
@EmmaVandewalle EmmaVandewalle linked an issue Mar 9, 2024 that may be closed by this pull request
Copy link
Contributor

@matt01y matt01y left a comment

Choose a reason for hiding this comment

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

Zou je mss ergens een // TODO: {{uitleg}} kunnen zetten zodat we snel kunnen vinden in code dat dit een tijdelijke tussenopls is?

Of maakt dit dat de lint test faalt?

@EmmaVandewalle
Copy link
Contributor Author

Zou je mss ergens een // TODO: {{uitleg}} kunnen zetten zodat we snel kunnen vinden in code dat dit een tijdelijke tussenopls is?

Of maakt dit dat de lint test faalt?

Ik zal straks eens kijken wat de linter dan zegt

@matt01y matt01y merged commit 6600b00 into main Mar 11, 2024
2 checks passed
@matt01y matt01y deleted the project-cards branch March 11, 2024 10:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

react projectcards aanmaken (student cards)
2 participants