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

[REFACTOR] Improved error handling #226

Merged
merged 12 commits into from
Sep 12, 2021
Merged

Conversation

Lissy93
Copy link
Owner

@Lissy93 Lissy93 commented Sep 12, 2021

Lissy93 /REFACTOR/improved-error-handling → Lissy93/dashy Commits: 12 | Files Changed: 43 | Additions: 204 🚚 Refactor Powered by Pull Request Badge

Category: Refactoring

Overview

  • Status logging function
  • Updated error handler
  • Disables service worker by default
  • Option to view error log in UI
  • Renamed and updated actions
  • Wrote docs for release cadence
  • Did some other insignificant stuff

Issue Number N/A

New Vars N/A

Screenshot N/A

Code Quality Checklist (Please complete)

  • All changes are backwards compatible
  • All lint checks and tests are passing
  • There are no (new) build warnings or errors
  • (If a new config option is added) Attribute is outlined in the schema and documented
  • (If a new dependency is added) Package is essential, and has been checked out for security or performance
  • Bumps version, if new feature added

@netlify
Copy link

netlify bot commented Sep 12, 2021

✔️ Deploy Preview for dashy-dev ready!

🔨 Explore the source changes: 97ddcb3

🔍 Inspect the deploy log: https://app.netlify.com/sites/dashy-dev/deploys/613e7314a0b2a70008adaf90

😎 Browse the preview: https://deploy-preview-226--dashy-dev.netlify.app/

@liss-bot
Copy link
Collaborator

Hi Lissy93! Thank you for contributing to Dashy! ✨

When making changes to the documentation, be sure to double check that:

  • Link and images URLs properly resolve
  • Your spelling and grammar is correct
  • Any markdown formatting is valid

Don't forget to verify they the config validator script responds correctly to your new attribute.


I'm a bot, and this is an automated comment 🤖

@viezly
Copy link

viezly bot commented Sep 12, 2021

This pull request is split into 6 parts for easier review.
👀 Review pull request on Viezly

Changed files are located in these folders:

  • .github
  • .github/workflows
  • /
  • docs
  • src/components
  • src/utils
  • src/views

@Lissy93 Lissy93 added the 🚚 Refactor [PR] Non-feature code-related updates and refactoring label Sep 12, 2021
@Lissy93 Lissy93 merged commit c0545f1 into master Sep 12, 2021
@Lissy93 Lissy93 deleted the REFACTOR/improved-error-handling branch September 12, 2021 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚚 Refactor [PR] Non-feature code-related updates and refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants