-
Notifications
You must be signed in to change notification settings - Fork 0
Feat/pin favorites #45
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In addition, you cannot unstar places now, neither from the list or from search unless you have not changed location after starring the place
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clicked the wrong thing, meant to request changes
|
Also, fix the formatting, but that should be obvious |
|
And the tests need to be updated to match the new contract, as you changed some component properties I reccomend running "npm run pre-commit" before commiting as it runs the linter, formatter and tests so you can avoid unnecessary extra commits to fix these issues |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice 🔥
favorites are pinned after refresh
må nok bruke state bs for å fikse automatisk pinning