-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[IMPROVMENTS] User-Requested Modifications #261
Conversation
…er-requested-modifications
✔️ Deploy Preview for dashy-dev ready! 🔨 Explore the source changes: e8b7ee1 🔍 Inspect the deploy log: https://app.netlify.com/sites/dashy-dev/deploys/6158b6ad84407000073df5b8 😎 Browse the preview: https://deploy-preview-261--dashy-dev.netlify.app |
Hi Lissy93! Thank you for contributing to Dashy! ✨ When making changes to the documentation, be sure to double check that:
Don't forget to verify they the config validator script responds correctly to your new attribute. Thank you for contributing to Dashy's themes 🎨
|
This pull request is split into 6 parts for easier review. Changed files are located in these folders:
|
This pull request fixes 2 alerts when merging c896120 into cf92331 - view on LGTM.com fixed alerts:
|
Size Change: +443 B (0%) Total Size: 1.89 MB
ℹ️ View Unchanged
|
This pull request fixes 2 alerts when merging e8b7ee1 into cf92331 - view on LGTM.com fixed alerts:
|
Closes Lissy93#261 Signed-off-by: Bjorn Lammers <[email protected]>
Category: Bugfix / Feature / Refactoring Only / Build related changes / Documentation
Overview
A whole bunch of small updates that were requested in recent issues
Issue Number: #163 #250 #254 #255 #259
New Vars
appConfig.workspaceLandingUrl
string
appConfig.colCount
number
1
and8
Screenshot N/A
Code Quality Checklist (Please complete)