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

Change the unit of the duration of a volunteer opportunity #559

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yashodha
Copy link
Collaborator

@yashodha yashodha commented Apr 9, 2021

Change the unit of the duration of a volunteer opportunity

@ginkgomzd
Copy link
Contributor

I think this would need an upgrade step for existing systems.

I believe this is a legacy rooted in how activities represent time, but I'd have to dig for what the precedent actually was.

I do know that the position has been that this is best handled by formatting in displays.

@yashodha
Copy link
Collaborator Author

yashodha commented Apr 12, 2021

@ginkgomzd I agree this was based on how we always show activity duration in minutes. But this PR is just a display improvement where we show the volunteer time in hours ( since it could be cumbersome to convert to minutes when you are logging 5/6 hours). I don't think any changes to the upgrade are warranted as it shouldn't affect how data is stored.

@ginkgomzd
Copy link
Contributor

ah, I didn't look carefully at what you were doing here.

Did you try to break it entering fractional hours?
How about converting from 20 minutes, 13 minutes...

I'll give this a proper look this week or next.

Cheers

@yashodha
Copy link
Collaborator Author

yashodha commented Jan 2, 2023

@ginkgomzd what do you think about this? Can you please take a look at this PR? Thanks!

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.

2 participants