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

[3.4] Refresh config checks page #6775

Merged
merged 3 commits into from Jul 6, 2017

Conversation

Projects
None yet
2 participants
@GawainLynch
Contributor

GawainLynch commented Jun 27, 2017

We've had this route, and the aysnc controller that isn't even used, since 3.0-dev, so this PR basically just adds it to the menu, and updates the admin route & templates to use bolt/requirements

Eye candy (normal UI team disclaimer):

checks-menu

checks-mixed

checks-all

Edit: The screenshot for #3602 (original PR) is, with hindsight, pretty funny; "Bolt 2.3.0 alpha 1" … 2 years ago

GawainLynch added some commits Jun 27, 2017

@GawainLynch GawainLynch added this to the Bolt 3.4 - Feature release milestone Jun 27, 2017

@bobdenotter

Looks good. I think this should be integrated into our 🔮 -questions, for when people have support questions, but didn't provide enough information.

@GawainLynch

This comment has been minimized.

Show comment
Hide comment
@GawainLynch

GawainLynch Jul 3, 2017

Contributor

Looks good. I think this should be integrated into our 🔮 -questions, for when people have support questions, but didn't provide enough information.

Cool, yeah good idea actually.

Also, this gives us one more trigger to get someone to review the requirement/recommendation list for bolt/requirements … I know some are 👍 and some will need 🔥

Contributor

GawainLynch commented Jul 3, 2017

Looks good. I think this should be integrated into our 🔮 -questions, for when people have support questions, but didn't provide enough information.

Cool, yeah good idea actually.

Also, this gives us one more trigger to get someone to review the requirement/recommendation list for bolt/requirements … I know some are 👍 and some will need 🔥

@bobdenotter

This comment has been minimized.

Show comment
Hide comment
@bobdenotter

bobdenotter Jul 6, 2017

Member

I know some are 👍 and some will need 🔥

Sure, but that's iterative. ;-)

Member

bobdenotter commented Jul 6, 2017

I know some are 👍 and some will need 🔥

Sure, but that's iterative. ;-)

@bobdenotter bobdenotter merged commit 2e05e77 into bolt:release/3.4 Jul 6, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@GawainLynch GawainLynch deleted the GawainLynch:hotfix/config-checks branch Jul 6, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment