DigitalExcellence / dex-frontend

Frontend for the Digital Excellence platform
https://dex.software
GNU Lesser General Public License v3.0
10 stars 5 forks source link

Bug/512 homepage like in background #567

Closed elislaats closed 3 years ago

elislaats commented 3 years ago

Description

Recommended project cards now update likes given on an open modal

Type of change

Checklist

Steps to Test or Reproduce

  1. Go to your homepage, make sure you have recommended projects (like some projects if you dont have recommended projects)
  2. Open your first recommended project
  3. Click like in the modal
  4. Check that the likes on the card in the background are updated too

Link to issue

Closes: #512

waltersajtos commented 3 years ago

Might be a good idea to implement this function on the overview page as well?

MeesvanStraten commented 3 years ago

@waltersajtos Yeah Julia implemented this also on the overview page 👍 Thanks!

waltersajtos commented 3 years ago

@MeesvanStraten @juliaslaats I'm pretty sure the overview page does not implement the modalUtitlity

elislaats commented 3 years ago

@MeesvanStraten @juliaslaats I'm pretty sure the overview page does not implement the modalUtitlity

You are right, the project overview has a separate modal-function, which is weird, I think it's an easy fix, I'll quickly look into it

MeesvanStraten commented 3 years ago

Hmm my bad, I only tested if it synced.

elislaats commented 3 years ago

That makes sense @MeesvanStraten, no problem!

I re-wrote the funtions on the overview page so that they use the modal utility aswell. Would you be so kind to check again that it works well. 😄

@waltersajtos I requested your review too if you happen to have some time. Thanks for pointing the issue out!