Skip to content
Snippets Groups Projects

See #872: banner setting

Merged See #872: banner setting
1 unresolved thread
Merged Agate requested to merge 872-banner-setting into develop
1 unresolved thread

See #872 (closed)

  • Add a configurable banner setting (API and admin UI)
  • Expose the banner URL in Nodeinfo

This isn't displayed anywhere yet, but this is a requirement for #872 (closed)

cc @funkwhale/reviewers-front @funkwhale/reviewers-python

Peek_2019-09-16_11-18

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
115 117 )
116 118 widget = widgets.Textarea
117 119 field_kwargs = {"required": False}
120
121
122 class ImageWidget(widgets.ClearableFileInput):
123 pass
124
125
126 class ImagePreference(types.FilePreference):
127 widget = ImageWidget
128 field_kwargs = {
129 "validators": [
130 FileExtensionValidator(allowed_extensions=["png", "jpg", "jpeg", "webp"])
  • Esteban
  • LGTM, but see comment about the test.

  • Agate added 6 commits

    added 6 commits

    Compare with previous version

  • Agate enabled an automatic merge when the pipeline for 6f79768d succeeds

    enabled an automatic merge when the pipeline for 6f79768d succeeds

  • merged

  • Agate mentioned in commit 1fd49e64

    mentioned in commit 1fd49e64

  • Please register or sign in to reply
    Loading