fix(weather_status): add all missing met.no symbol codes to weatherOptions#59317
Conversation
|
@psimaker thank you for your pull request! |
44727c1 to
701baf9
Compare
|
@susnux - Done — rebased onto current master. Sorry for the merge commit, lesson learned! |
|
@psimaker this looks good! npm ci
npm run build |
…tions Signed-off-by: psimaker <psimaker@users.noreply.github.com>
Signed-off-by: psimaker <psimaker@users.noreply.github.com>
701baf9 to
19b0c91
Compare
done. rebased onto current master and compiled frontend assets. |
|
/backport af8cb33 to stable32 |
|
/backport af8cb33 to stable33 |
|
Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22 |
Fixes #59292
The
weatherOptionsobject inapps/weather_status/src/App.vueonly mapped 25 of 83 validsymbol_codevalues from the met.no Locationforecast API. Any unmapped code caused the widget to show "Unknown weather code".This PR adds the remaining 58 codes following the existing pattern. All icons were already in place as confirmed by @joshtrichards.