Merge pull request #56794 from nextcloud/feat/hide-developer-docs-conditionally

docs(settings): Removing the developerDocsUrl
pull/57658/head
Côme Chilliet 4 months ago committed by GitHub
commit 433e3da4d3
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
  1. 1
      apps/settings/lib/Controller/AppSettingsController.php
  2. 5
      apps/settings/src/views/AppStoreNavigation.vue
  3. 4
      apps/settings/tests/Controller/AppSettingsControllerTest.php
  4. 4
      dist/settings-apps-view-4529.js
  5. 2
      dist/settings-apps-view-4529.js.map
  6. 4
      dist/settings-vue-settings-apps-users-management.js
  7. 2
      dist/settings-vue-settings-apps-users-management.js.map

@ -92,7 +92,6 @@ class AppSettingsController extends Controller {
$this->initialState->provideInitialState('appstoreEnabled', $this->config->getSystemValueBool('appstoreenabled', true));
$this->initialState->provideInitialState('appstoreBundles', $this->getBundles());
$this->initialState->provideInitialState('appstoreDeveloperDocs', $this->urlGenerator->linkToDocs('developer-manual'));
$this->initialState->provideInitialState('appstoreUpdateCount', count($this->getAppsWithUpdates()));
if ($this->appManager->isEnabledForAnyone('app_api')) {

@ -100,10 +100,6 @@
</NcAppNavigationItem>
</template>
<NcAppNavigationItem
id="app-developer-docs"
:name="t('settings', 'Developer documentation ↗')"
:href="developerDocsUrl" />
</template>
</NcAppNavigation>
</template>
@ -123,7 +119,6 @@ import APPSTORE_CATEGORY_ICONS from '../constants/AppstoreCategoryIcons.ts'
import { useAppsStore } from '../store/apps-store.ts'
const appstoreEnabled = loadState<boolean>('settings', 'appstoreEnabled', true)
const developerDocsUrl = loadState<string>('settings', 'appstoreDeveloperDocs', '')
const store = useAppsStore()
const categories = computed(() => store.categories)

@ -170,7 +170,7 @@ class AppSettingsControllerTest extends TestCase {
->with('core_apps');
$this->initialState
->expects($this->exactly(4))
->expects($this->exactly(3))
->method('provideInitialState');
$policy = new ContentSecurityPolicy();
@ -203,7 +203,7 @@ class AppSettingsControllerTest extends TestCase {
->with('core_apps');
$this->initialState
->expects($this->exactly(4))
->expects($this->exactly(3))
->method('provideInitialState');
$policy = new ContentSecurityPolicy();

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long
Loading…
Cancel
Save