-
-
Notifications
You must be signed in to change notification settings - Fork 21
Closed
Description
Due diligence
- I have searched all open and closed issues for this problem.
- I have searched all open and closed discussions for this problem.
- I have looked at the CHANGELOG to make sure this problem has not already been addressed.
- I agree to the Code of Conduct of this repository
Describe the bug
at a minimum there should be a notice/warning displayed on the view=scannertasks and view=backuptasks
Reproduction instructions
for a site which does not have AB or AT set up an automation
Expected behaviour
info that it is not possible for the automation to run on the site as AB/AT are not installed etc
Actual behaviour
no message
Screenshots
No response
Panopticon version
git
PHP version
8.2
Database type
MySQL
Database version
8
Browser
None
Browser version
No response
Additional information
I thought we could use the same code as used here
panopticon/ViewTemplates/Sites/item_admintools.blade.php
Lines 91 to 102 in 3542299
| <div class="card-body collapse show" id="cardAdminToolsBody"> | |
| @if ($this->hasAdminTools && !$this->hasAdminToolsPro) | |
| @yield('atErrorNotPro') | |
| @elseif(!$this->hasAdminTools) | |
| @yield('atErrorNotInstalled') | |
| @elseif(!$this->siteConfig->get('core.admintools.enabled')) | |
| @yield('atErrorDisabled') | |
| @else | |
| @include('Sites/item_admintools_actions') | |
| @include('Sites/item_admintools_scanner') | |
| @endif | |
| </div> |
But I tried and failed without a lot of code duplication which I'm sure is not right anyway
Metadata
Metadata
Assignees
Labels
No labels