Remaining tasks

  • Add a configuration route in cloudinary_sdk.info.yml
  • Set up new submodule cloudinary_media_library_widget
  • Add the following settings to the form
    • SAML Authentication (checkbox, under the widget fieldset, optional).
    • Starting folder (select, single, under the widget fieldset, optional). Allowed values are the same as for Cloudinary scheme with root folder (values are not available if the API is not configured yet)
    • Optimizations (fieldset)
    • Default optimizations for image (textarea, under the optimizations fieldset, optional). The stored value should be an array (item per new line)
    • Default optimizations for video (textarea, under the optimizations fieldset, optional). The stored value should be an array (item per new line)
    • Default DPR (number, step 0.1, under the optimizations fieldset, optional)
    • Responsive breakpoints (textarea, under the optimizations fieldset, optional). The stored value should be an array (item per new line)
    • Video playback (fieldset)
    • Default player (radios, under the video fieldset, required). Allowed values are
      HTML 5 Player
      Cloudinary Video player
      
    • Default player configuration (textarea, json or yaml editor attached, optional, available when cloudinary player is chosen)
  • Provide clear description for each extra setting from the docs
  • Init default config cloudinary_media_library_widget.settings.yml in config/install
    • Use default values for the widget settings
    • The default value for the image optimization is
      - f_auto
      - q_auto
      
  • Init config schema cloudinary_media_library_widget.schema.yml in config/schema
    • Describe the widget settings

Issue fork cloudinary-3282657

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nginex created an issue. See original summary.

sdstyles’s picture

Assigned: Unassigned » sdstyles

nginex’s picture

Assigned: sdstyles » nginex
Status: Active » Needs review

  • 76e2a27 committed on 2.x
    Issue #3282657: Refactor form, replace old images.
    
nginex’s picture

Assigned: nginex » Unassigned
Status: Needs review » Fixed

I did refactor some code related to the form. I also provided a custom helper service to use instead of functions.

nginex’s picture

Issue tags: +D9 Porting

sdstyles’s picture

Assigned: Unassigned » nginex
Status: Fixed » Needs review

  • 1038a1d committed on 2.x
    Issue #3282657: Improve config form. Seperate stream wrapper configs.
    
  • sdstyles authored 521f50d on 2.x
    Issue #3282657 Create cloudinary_video module and move video...
  • sdstyles authored 605724b on 2.x
    Issue #3282657 Remove video config, store image optimizations as text...
  • sdstyles authored cbce5c3 on 2.x
    Issue #3282657 change 'API secret' field type to password
    
nginex’s picture

Assigned: nginex » Unassigned
Status: Needs review » Fixed

Improved SDK config form. Separated configs for stream wrapper module.

nginex’s picture

Assigned: Unassigned » sdstyles
Status: Fixed » Needs work

Some extra changes are needed like adding CNAME option, starting folder integration, and improving the UI for the widget.

  • 10fbfc9 committed on 2.x
    Issue #3282657: Add cname option.
    
nginex’s picture

Assigned: sdstyles » Unassigned
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.