nextcloud / nextcloud/cookbook
Add error handling for all responses in `AppNavi.vue`
Open
Nobody has claimed this yet.
Frontend
- Dominant language
- HTML
- Stars
- 641
- Forks
- 113
- Avg merge
- 21h 31m
- Merged PRs (30d)
- 26
Description
There are some axios calls in AppNavi.vue which don’t check the response.
We should add some code to check the response and handle any possible errors (this was shortly discussed in #497).
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating AppNavi.vue and inventorying its axios calls that do not check responses. Review the discussion in #497 for the intended handling, then ensure every relevant response and possible error is addressed; done means no axios call in that component lacks response handling.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100