diff --git a/api/funkwhale_api/common/dynamic_preferences_registry.py b/api/funkwhale_api/common/dynamic_preferences_registry.py index 2374de7c7..15b182671 100644 --- a/api/funkwhale_api/common/dynamic_preferences_registry.py +++ b/api/funkwhale_api/common/dynamic_preferences_registry.py @@ -16,5 +16,5 @@ class APIAutenticationRequired( help_text = ( 'If disabled, anonymous users will be able to query the API' 'and access music data (as well as other data exposed in the API ' - 'without specific permissions)' + 'without specific permissions).' ) diff --git a/api/funkwhale_api/federation/dynamic_preferences_registry.py b/api/funkwhale_api/federation/dynamic_preferences_registry.py index e86b9f6f2..8b1b2b03f 100644 --- a/api/funkwhale_api/federation/dynamic_preferences_registry.py +++ b/api/funkwhale_api/federation/dynamic_preferences_registry.py @@ -19,6 +19,9 @@ class MusicCacheDuration(types.IntPreference): 'locally? Federated files that were not listened in this interval ' 'will be erased and refetched from the remote on the next listening.' ) + field_kwargs = { + 'required': False, + } @global_preferences_registry.register @@ -29,7 +32,7 @@ class Enabled(preferences.DefaultFromSettingMixin, types.BooleanPreference): verbose_name = 'Federation enabled' help_text = ( 'Use this setting to enable or disable federation logic and API' - ' globally' + ' globally.' ) @@ -41,8 +44,11 @@ class CollectionPageSize( setting = 'FEDERATION_COLLECTION_PAGE_SIZE' verbose_name = 'Federation collection page size' help_text = ( - 'How much items to display in ActivityPub collections' + 'How much items to display in ActivityPub collections.' ) + field_kwargs = { + 'required': False, + } @global_preferences_registry.register @@ -54,8 +60,11 @@ class ActorFetchDelay( verbose_name = 'Federation actor fetch delay' help_text = ( 'How much minutes to wait before refetching actors on ' - 'request authentication' + 'request authentication.' ) + field_kwargs = { + 'required': False, + } @global_preferences_registry.register @@ -66,6 +75,6 @@ class MusicNeedsApproval( setting = 'FEDERATION_MUSIC_NEEDS_APPROVAL' verbose_name = 'Federation music needs approval' help_text = ( - 'When true, other federation actors will require your approval' + 'When true, other federation actors will need your approval' ' before being able to browse your library.' ) diff --git a/api/funkwhale_api/instance/dynamic_preferences_registry.py b/api/funkwhale_api/instance/dynamic_preferences_registry.py index 20679fd3d..8ccf80dd9 100644 --- a/api/funkwhale_api/instance/dynamic_preferences_registry.py +++ b/api/funkwhale_api/instance/dynamic_preferences_registry.py @@ -13,8 +13,11 @@ class InstanceName(types.StringPreference): section = instance name = 'name' default = '' - help_text = 'Instance public name' - verbose_name = 'The public name of your instance' + verbose_name = 'Public name' + help_text = 'The public name of your instance, displayed in the about page.' + field_kwargs = { + 'required': False, + } @global_preferences_registry.register @@ -23,7 +26,11 @@ class InstanceShortDescription(types.StringPreference): section = instance name = 'short_description' default = '' - verbose_name = 'Instance succinct description' + verbose_name = 'Short description' + help_text = 'Instance succinct description, displayed in the about page.' + field_kwargs = { + 'required': False, + } @global_preferences_registry.register @@ -31,31 +38,31 @@ class InstanceLongDescription(types.StringPreference): show_in_api = True section = instance name = 'long_description' + verbose_name = 'Long description' default = '' - help_text = 'Instance long description (markdown allowed)' + help_text = 'Instance long description, displayed in the about page (markdown allowed).' + widget = widgets.Textarea field_kwargs = { - 'widget': widgets.Textarea + 'required': False, } + @global_preferences_registry.register class RavenDSN(types.StringPreference): show_in_api = True section = raven name = 'front_dsn' default = 'https://9e0562d46b09442bb8f6844e50cbca2b@sentry.eliotberriot.com/4' - verbose_name = ( - 'A raven DSN key used to report front-ent errors to ' - 'a sentry instance' - ) + verbose_name = 'Raven DSN key (front-end)' + help_text = ( - 'Keeping the default one will report errors to funkwhale developers' + 'A Raven DSN key used to report front-ent errors to ' + 'a sentry instance. Keeping the default one will report errors to ' + 'Funkwhale developers.' ) - - -SENTRY_HELP_TEXT = ( - 'Error reporting is disabled by default but you can enable it if' - ' you want to help us improve funkwhale' -) + field_kwargs = { + 'required': False, + } @global_preferences_registry.register @@ -65,8 +72,7 @@ class RavenEnabled(types.BooleanPreference): name = 'front_enabled' default = False verbose_name = ( - 'Wether error reporting to a Sentry instance using raven is enabled' - ' for front-end errors' + 'Report front-end errors with Raven' ) @@ -78,7 +84,7 @@ class InstanceNodeinfoEnabled(types.BooleanPreference): default = True verbose_name = 'Enable nodeinfo endpoint' help_text = ( - 'This endpoint is needed for your about page to work.' + 'This endpoint is needed for your about page to work. ' 'It\'s also helpful for the various monitoring ' 'tools that map and analyzize the fediverse, ' 'but you can disable it completely if needed.' @@ -91,10 +97,10 @@ class InstanceNodeinfoPrivate(types.BooleanPreference): section = instance name = 'nodeinfo_private' default = False - verbose_name = 'Enable nodeinfo endpoint' + verbose_name = 'Private mode in nodeinfo' help_text = ( - 'Indicate in the nodeinfo endpoint that you do not want your instance' - 'to be tracked by third-party services.' + 'Indicate in the nodeinfo endpoint that you do not want your instance ' + 'to be tracked by third-party services. ' 'There is no guarantee these tools will honor this setting though.' ) @@ -107,6 +113,6 @@ class InstanceNodeinfoStatsEnabled(types.BooleanPreference): default = True verbose_name = 'Enable usage and library stats in nodeinfo endpoint' help_text = ( - 'Disable this f you don\'t want to share usage and library statistics' + 'Disable this if you don\'t want to share usage and library statistics ' 'in the nodeinfo endpoint but don\'t want to disable it completely.' ) diff --git a/api/funkwhale_api/instance/urls.py b/api/funkwhale_api/instance/urls.py index f506488fc..7992842c0 100644 --- a/api/funkwhale_api/instance/urls.py +++ b/api/funkwhale_api/instance/urls.py @@ -1,9 +1,11 @@ from django.conf.urls import url +from rest_framework import routers from . import views - +admin_router = routers.SimpleRouter() +admin_router.register(r'admin/settings', views.AdminSettings, 'admin-settings') urlpatterns = [ url(r'^nodeinfo/2.0/$', views.NodeInfo.as_view(), name='nodeinfo-2.0'), url(r'^settings/$', views.InstanceSettings.as_view(), name='settings'), -] +] + admin_router.urls diff --git a/api/funkwhale_api/instance/views.py b/api/funkwhale_api/instance/views.py index 5953ca555..e6725e248 100644 --- a/api/funkwhale_api/instance/views.py +++ b/api/funkwhale_api/instance/views.py @@ -2,6 +2,7 @@ from rest_framework import views from rest_framework.response import Response from dynamic_preferences.api import serializers +from dynamic_preferences.api import viewsets as preferences_viewsets from dynamic_preferences.registries import global_preferences_registry from funkwhale_api.common import preferences @@ -15,6 +16,10 @@ NODEINFO_2_CONTENT_TYPE = ( ) +class AdminSettings(preferences_viewsets.GlobalPreferencesViewSet): + pagination_class = None + + class InstanceSettings(views.APIView): permission_classes = [] authentication_classes = [] diff --git a/api/funkwhale_api/playlists/dynamic_preferences_registry.py b/api/funkwhale_api/playlists/dynamic_preferences_registry.py index 21140fa14..b717177a2 100644 --- a/api/funkwhale_api/playlists/dynamic_preferences_registry.py +++ b/api/funkwhale_api/playlists/dynamic_preferences_registry.py @@ -13,3 +13,6 @@ class MaxTracks(preferences.DefaultFromSettingMixin, types.IntegerPreference): name = 'max_tracks' verbose_name = 'Max tracks per playlist' setting = 'PLAYLISTS_MAX_TRACKS' + field_kwargs = { + 'required': False, + } diff --git a/api/funkwhale_api/providers/acoustid/dynamic_preferences_registry.py b/api/funkwhale_api/providers/acoustid/dynamic_preferences_registry.py index da785df40..33c9643b0 100644 --- a/api/funkwhale_api/providers/acoustid/dynamic_preferences_registry.py +++ b/api/funkwhale_api/providers/acoustid/dynamic_preferences_registry.py @@ -1,3 +1,5 @@ +from django import forms + from dynamic_preferences.types import StringPreference, Section from dynamic_preferences.registries import global_preferences_registry @@ -11,3 +13,7 @@ class APIKey(StringPreference): default = '' verbose_name = 'Acoustid API key' help_text = 'The API key used to query AcoustID. Get one at https://acoustid.org/new-application.' + widget = forms.PasswordInput + field_kwargs = { + 'required': False, + } diff --git a/api/funkwhale_api/providers/youtube/dynamic_preferences_registry.py b/api/funkwhale_api/providers/youtube/dynamic_preferences_registry.py index fc7f7d793..ac5fc4bde 100644 --- a/api/funkwhale_api/providers/youtube/dynamic_preferences_registry.py +++ b/api/funkwhale_api/providers/youtube/dynamic_preferences_registry.py @@ -1,3 +1,5 @@ +from django import forms + from dynamic_preferences.types import StringPreference, Section from dynamic_preferences.registries import global_preferences_registry @@ -11,3 +13,7 @@ class APIKey(StringPreference): default = 'CHANGEME' verbose_name = 'YouTube API key' help_text = 'The API key used to query YouTube. Get one at https://console.developers.google.com/.' + widget = forms.PasswordInput + field_kwargs = { + 'required': False, + } diff --git a/api/funkwhale_api/users/dynamic_preferences_registry.py b/api/funkwhale_api/users/dynamic_preferences_registry.py index 16d79da14..4f7360530 100644 --- a/api/funkwhale_api/users/dynamic_preferences_registry.py +++ b/api/funkwhale_api/users/dynamic_preferences_registry.py @@ -10,6 +10,7 @@ class RegistrationEnabled(types.BooleanPreference): section = users name = 'registration_enabled' default = False - verbose_name = ( - 'Can visitors open a new account on this instance?' + verbose_name = 'Open registrations to new users' + help_text = ( + 'When enabled, new users will be able to register on this instance.' ) diff --git a/api/funkwhale_api/users/models.py b/api/funkwhale_api/users/models.py index f067a2a8b..8273507c4 100644 --- a/api/funkwhale_api/users/models.py +++ b/api/funkwhale_api/users/models.py @@ -6,7 +6,7 @@ import os import uuid from django.conf import settings -from django.contrib.auth.models import AbstractUser +from django.contrib.auth.models import AbstractUser, Permission from django.urls import reverse from django.db import models from django.utils.encoding import python_2_unicode_compatible @@ -55,6 +55,10 @@ class User(AbstractUser): def __str__(self): return self.username + def add_permission(self, codename): + p = Permission.objects.get(codename=codename) + self.user_permissions.add(p) + def get_absolute_url(self): return reverse('users:detail', kwargs={'username': self.username}) diff --git a/api/tests/instance/test_views.py b/api/tests/instance/test_views.py index 468c0ddae..6d8dcac3e 100644 --- a/api/tests/instance/test_views.py +++ b/api/tests/instance/test_views.py @@ -21,3 +21,31 @@ def test_nodeinfo_endpoint_disabled(db, api_client, preferences): response = api_client.get(url) assert response.status_code == 404 + + +def test_settings_only_list_public_settings(db, api_client, preferences): + url = reverse('api:v1:instance:settings') + response = api_client.get(url) + + for conf in response.data: + p = preferences.model.objects.get( + section=conf['section'], name=conf['name']) + assert p.preference.show_in_api is True + + +def test_admin_settings_restrict_access(db, logged_in_api_client, preferences): + url = reverse('api:v1:instance:admin-settings-list') + response = logged_in_api_client.get(url) + + assert response.status_code == 403 + + +def test_admin_settings_correct_permission( + db, logged_in_api_client, preferences): + user = logged_in_api_client.user + user.add_permission('change_globalpreferencemodel') + url = reverse('api:v1:instance:admin-settings-list') + response = logged_in_api_client.get(url) + + assert response.status_code == 200 + assert len(response.data) == len(preferences.all()) diff --git a/changes/changelog.d/206.feature b/changes/changelog.d/206.feature new file mode 100644 index 000000000..b334554fe --- /dev/null +++ b/changes/changelog.d/206.feature @@ -0,0 +1,17 @@ +We now have a brand new instance settings interface in the front-end (#206) + + +Instance settings interface +^^^^^^^^^^^^^^^^^^^^^^^^^^^ + +Prior to this release, the only way to update instance settings (such as +instance description, signup policy, federation configuration, etc.) was using +the admin interface provided by Django (the back-end framework which power the API). + +This interface worked, but was not really-user friendly and intuitive. + +Starting from this release, we now offer a dedicated interface directly +in the front-end. You can view and edit all your instance settings from here, +assuming you have the required permissions. + +This interface is available at ``/manage/settings` and via link in the sidebar. diff --git a/docs/configuration.rst b/docs/configuration.rst index bbc658e08..b7df2db42 100644 --- a/docs/configuration.rst +++ b/docs/configuration.rst @@ -27,15 +27,24 @@ Those settings are stored in database and do not require a restart of your instance after modification. They typically relate to higher level configuration, such your instance description, signup policy and so on. -There is no polished interface for those settings, yet, but you can view update -them using the administration interface provided by Django (the framework funkwhale is built on). - -The URL should be ``/api/admin/dynamic_preferences/globalpreferencemodel/`` (prepend your domain in front of it, of course). +You can edit those settings directly from the web application, assuming +you have the required permissions. The URL is ``/manage/settings``, and +you will also find a link to this page in the sidebar. If you plan to use acoustid and external imports (e.g. with the youtube backends), you should edit the corresponding settings in this interface. +.. note:: + + If you have any issue with the web application, a management interface is also + available for those settings from Django's administration interface. It's + less user friendly, though, and we recommend you use the web app interface + whenever possible. + + The URL should be ``/api/admin/dynamic_preferences/globalpreferencemodel/`` (prepend your domain in front of it, of course). + + Configuration reference ----------------------- diff --git a/front/src/components/About.vue b/front/src/components/About.vue index 524191250..b0ae67ef7 100644 --- a/front/src/components/About.vue +++ b/front/src/components/About.vue @@ -13,6 +13,12 @@
{{ $t('Unfortunately, owners of this instance did not yet take the time to complete this page.') }}
+