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

fix(project): update follower count client side #1038

Merged
merged 2 commits into from
Mar 26, 2023
Merged

fix(project): update follower count client side #1038

merged 2 commits into from
Mar 26, 2023

Conversation

MagnusHJensen
Copy link
Contributor

Description:
Ensures the project followers on the client side is in-sync with the actions taken.

Media:

Screen.Recording.2023-03-11.at.11.59.42.mov

Closes: modrinth/code#1912

Copy link
Contributor

@triphora triphora left a comment

Choose a reason for hiding this comment

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

Wouldn't it be easier to add it to user(un)followProject in composables/user.js?

@MagnusHJensen
Copy link
Contributor Author

Maybe, I just Saw the composables files as they should only have Logic relater to the name/file so user in this case, where incrementing and decrementing is not user related but more page specific in My mind.
I can easily change it if I misunderstood how they are separated :D

@triphora
Copy link
Contributor

tbh, I don't know how it's "supposed to be" separated. In any event, incrementing and decrementing is going to be wanted no matter which page it's on.

@MagnusHJensen
Copy link
Contributor Author

Moved to user composable

@Geometrically Geometrically merged commit 25bd615 into modrinth:master Mar 26, 2023
@MagnusHJensen MagnusHJensen deleted the fix/follower-count-not-updating-visually branch March 27, 2023 11:10
Morb0 pushed a commit to CorvaxStation/knossos that referenced this pull request Jun 30, 2023
* fix(project): update follower count client side

* move following increment to user composable

---------

Co-authored-by: MagnusHJensen <magnus.holm.jensen@lego.dk>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Following projects doesn't increment client side follow count
3 participants