Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pull request door marijn #49

Open
wants to merge 75 commits into
base: marijn
Choose a base branch
from
Open

Pull request door marijn #49

wants to merge 75 commits into from

Conversation

MarijnMolenaar
Copy link
Owner

No description provided.

Joosiii and others added 30 commits March 29, 2022 21:32
Push de applicatie van Joost (+ navbar update, registratie mailfunctie, matches pagina en likes pagina) naar onze eigen repo
Voegt filterfunctie toe, zit echter nog een bug in die ik niet opgelost krijg, wanneer je filtert kom je op een errorpagina terecht.
De Express dependency was ineens verdwenen uit de package.json, dus die is weer toegevoegd. Ook is een kleine bug gefixt omtrent de country dropdown in het filter scherm.
Fix van de bug wanneer de filters toepassen knop ingedrukt werd, was gewoon een simpele / vergeten... :)
- Add the dislike feature, update database.
- Fixes the matches page, you can now see only one profile, after liking or disliking you see a new one.
- Added the active state in the nav-bar for better User Experience
Add Dislike Feature + Fix Match Screen + Add Active State
When clicking the arrow button next to a liked profile you get redirected to a page where you can view the profile.
multer
error handling
Header sticky gemaakt
Joosiii and others added 30 commits April 7, 2022 19:32
Added Styling / Changed HTML / Added Sessions
Added Session Secret
+ Gulp Final Version + Bundling Last version CSS & JS
Nodemailer Security + App Fix + Removing Comments
Style.css toegevoegd aan de hand van marijns aanpassingen
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants