Skip to content

[configuration] Add image uploader for configuration logos - #11014

Open
Montekkundan wants to merge 1 commit into
aces:mainfrom
Montekkundan:montek/10812-configuration-uploader
Open

[configuration] Add image uploader for configuration logos#11014
Montekkundan wants to merge 1 commit into
aces:mainfrom
Montekkundan:montek/10812-configuration-uploader

Conversation

@Montekkundan

Copy link
Copy Markdown
Contributor

Brief summary of changes

  • Adds an image configuration type for study and login logos.
  • Adds a configuration upload endpoint using the existing LORIS file upload handler.
  • Validates uploaded file names, extensions, and image content.
  • Lists valid images from htdocs/images in the configuration selectors.
  • Adds a configurable upload directory and corresponding database patch.
  • Updates the Raisinbread configuration data.
  • Adds unit tests for image discovery, validation, and public paths.

Testing instructions

  • Open http://localhost:8080/configuration/ as a user with the config permission.
  • Confirm the study, login, and partner logo settings display image selectors.
  • Upload a PNG, JPEG, GIF, or WebP image.
  • Confirm the uploaded image appears in each image selector.
  • Select the uploaded image, submit the form, and reload the page.
  • Confirm the selected image remains saved and is publicly accessible.
  • Attempt to upload a non-image file and confirm it is rejected.
  • Add a Partner Logos field and confirm it starts with a blank selection.
  • Run target=configuration npm run compile.

Link(s) to related issue(s)

@Montekkundan Montekkundan self-assigned this Jul 25, 2026
@github-actions github-actions Bot added Language: SQL PR or issue that update SQL code RaisinBread PR or issue introducing/requiring improvements to the Raidinbread dataset Language: PHP PR or issue that update PHP code Language: Javascript PR or issue that update Javascript code Module: configuration PR or issue related to configuration module labels Jul 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language: Javascript PR or issue that update Javascript code Language: PHP PR or issue that update PHP code Language: SQL PR or issue that update SQL code Module: configuration PR or issue related to configuration module RaisinBread PR or issue introducing/requiring improvements to the Raidinbread dataset

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[configuration] Uploader

1 participant