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.

Sign up for GitHub

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

Jump to bottom

[weather_status] Snowfall #30551

Closed
Valdnet opened this issue Jan 8, 2022 · 4 comments · Fixed by #48353
Closed

[weather_status] Snowfall #30551

Valdnet opened this issue Jan 8, 2022 · 4 comments · Fixed by #48353
Labels
3. to review Waiting for reviews enhancement feature: dashboard good first issue Small tasks with clear documentation about how and in which place you need to fix things in.

Comments

@Valdnet
Copy link
Contributor

Valdnet commented Jan 8, 2022 •
edited
Loading

@eneiluj There is rainfall in "weather status". What about snowfall?

For meteorologists, these are two different types of precipitation.
@Valdnet Valdnet added the 0. Needs triage Pending check for reproducibility or if it fits our roadmap label Jan 8, 2022
@joshtrichards
Copy link
Member

Breadcrumbs for future work (by someone):

https://docs.api.met.no/doc/weathericons.html

https://github.com/metno/weather_symbol

server/apps/weather_status/src/App.vue

Line 101 in a62a8fa

const weatherOptions = {

@joshtrichards joshtrichards added feature: dashboard good first issue Small tasks with clear documentation about how and in which place you need to fix things in. labels Aug 18, 2024
@XxTysweezyxX
Copy link
Contributor

I'd like to work on this issue if possible

@joshtrichards
Copy link
Member

Hi @XxTysweezyxX - Absolutely. Have at it. :-)

XxTysweezyxX added a commit to XxTysweezyxX/server that referenced this issue Sep 3, 2024
@XxTysweezyxX
Fix nextcloud#30551: Added snowfall option to weather status
b772699
@XxTysweezyxX
Copy link
Contributor

could i get some feedback on the code?

@joshtrichards joshtrichards added 3. to review Waiting for reviews and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Sep 5, 2024
@joshtrichards joshtrichards mentioned this issue Sep 5, 2024
Closed
5 tasks
julien-nc pushed a commit that referenced this issue Sep 25, 2024
@XxTysweezyxX @julien-nc
Fix #30551: Added snowfall option to weather status
3bd3574
julien-nc pushed a commit that referenced this issue Sep 25, 2024
@XxTysweezyxX @julien-nc
Fix #30551: Added snowfall option to weather status
8d644ca
@julien-nc julien-nc mentioned this issue Sep 25, 2024
Merged
julien-nc pushed a commit that referenced this issue Sep 25, 2024
@XxTysweezyxX @julien-nc
Fix #30551: Added snowfall option to weather status
ea9e7fa 
Signed-off-by: Julien Veyssier <[email protected]>
julien-nc pushed a commit that referenced this issue Sep 25, 2024
@XxTysweezyxX @julien-nc
Fix #30551: Added snowfall option to weather status
1826ba2 
Signed-off-by: Julien Veyssier <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
3. to review Waiting for reviews enhancement feature: dashboard good first issue Small tasks with clear documentation about how and in which place you need to fix things in.
Projects
None yet
Milestone
No milestone
3 participants
@joshtrichards @Valdnet @XxTysweezyxX