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

Export Reward Point Summary #2020

Closed
janno42 opened this issue Sep 28, 2023 · 5 comments · Fixed by #2216
Closed

Export Reward Point Summary #2020

janno42 opened this issue Sep 28, 2023 · 5 comments · Fixed by #2216
Assignees
Labels
[C] Backend Focuses on backend implementation [C] Frontend Focuses on frontend implementation [P] Major Major priority [S] Easy This issue is self-contained and does not touch too many parts. Good choice for beginners.
Milestone

Comments

@janno42
Copy link
Member

janno42 commented Sep 28, 2023

A new button Export reward points should be added to the right of the text showing the number of available reward points on the staff reward points redemption events page. Clicking the button should download a CSV file containing a summary of the reward points.

This file should contain the two columns Email and Points for each user, listing the number of points currently available for that user (grantings minus redemptions) next to the user's email address. A line should only be added for users where this number of available points is not zero.

@janno42 janno42 added [C] Backend Focuses on backend implementation [C] Frontend Focuses on frontend implementation [P] Major Major priority [S] Easy This issue is self-contained and does not touch too many parts. Good choice for beginners. labels Sep 28, 2023
@janno42 janno42 added this to the Winter 2023 milestone Sep 28, 2023
@joendter
Copy link
Collaborator

joendter commented Oct 16, 2023

for the sake of consistency i think this should either result in a xls file or the Export Redemptions button should output a csv as well
It also should have consistent content: Export Redemptions gives Last name, First name, Email address and Number of points, not only Email address and points

@niklasmohrin
Copy link
Member

We talked about this shortly in person and I agree partially: Since both sheets seem to be more on the "just data" than the "this is a spreadsheet with colors and stuff like that" side, I would even say that both should be CSV (as Excel can just open the CSV too). @janno42 What do you think?

@richardebeling
Copy link
Member

@joendter do you want to continue working on this or should we unassign you?

@janno42 janno42 modified the milestones: Winter 2023, Summer 2024 Feb 26, 2024
@joendter
Copy link
Collaborator

Go ahead and unassign me

@janno42
Copy link
Member Author

janno42 commented May 27, 2024

We talked about this shortly in person and I agree partially: Since both sheets seem to be more on the "just data" than the "this is a spreadsheet with colors and stuff like that" side, I would even say that both should be CSV (as Excel can just open the CSV too). @janno42 What do you think?

I'd say CSV and just email and points is sufficient for the use case of this issue. The exported file is used internally for a quick overview of the current reward point status and we don't need the name information because the export is not used for, e.g., email generation. I also prefer CSV over Excel where we don't need formatting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C] Backend Focuses on backend implementation [C] Frontend Focuses on frontend implementation [P] Major Major priority [S] Easy This issue is self-contained and does not touch too many parts. Good choice for beginners.
Development

Successfully merging a pull request may close this issue.

5 participants