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

fix: Fix Display of Kudos List from Oveview Widget - MEED-7741 - Meeds-io/meeds#2559 - Meeds-io/meeds#1768 #548

Merged
merged 1 commit into from
Nov 1, 2024

Conversation

boubaker
Copy link
Member

@boubaker boubaker commented Nov 1, 2024

Prior to this change, the list of received/sent Kudos was retrieved with an empty period which will retrieve the configured default period (WEEK). This change ensures to propagate the periodType into the list of Kudos to display in order to fix the listing. At the same time, this will fix a glich on sending kudos button to add a loading effect.

…s-io/meeds#2559 - Meeds-io/meeds#1768

Prior to this change, the list of received/sent Kudos was retrieved with an empty period which will retrieve the configured default period (WEEK). This change ensures to propagate the periodType into the list of Kudos to display in order to fix the listing. At the same time, this will fix a glich on sending kudos button to add a loading effect.
@boubaker boubaker merged commit 89b83d3 into feature/whitepaper Nov 1, 2024
2 of 3 checks passed
@boubaker boubaker deleted the meed-7741 branch November 1, 2024 11:09
Copy link

sonarcloud bot commented Nov 1, 2024

boubaker added a commit that referenced this pull request Nov 1, 2024
…s-io/meeds#2559 - Meeds-io/meeds#1768 (#548)

Prior to this change, the list of received/sent Kudos was retrieved with
an empty period which will retrieve the configured default period
(WEEK). This change ensures to propagate the periodType into the list of
Kudos to display in order to fix the listing. At the same time, this
will fix a glich on sending kudos button to add a loading effect.
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.

1 participant