Skip to content

web: centralize file picker custom values#23169

Open
dominic-r wants to merge 1 commit into
mainfrom
sdko/file-picker-custom-values
Open

web: centralize file picker custom values#23169
dominic-r wants to merge 1 commit into
mainfrom
sdko/file-picker-custom-values

Conversation

@dominic-r

Copy link
Copy Markdown
Member

Summary

  • Adds a typed createObject hook to ak-search-select so generic creatable behavior no longer assumes custom objects look like { name: value }.
  • Moves file-picker-specific custom value construction into ak-file-search-input.
  • Shares in-flight file-list requests by usage and query so pages with multiple file pickers do not issue duplicate initial adminFileList calls.

Why

The file picker needs to support custom values like external URLs and Font Awesome icons, but that object shape belongs to the file picker, not the generic search-select. Consolidating that behavior keeps all file-picker surfaces consistent and reduces duplicate media-list requests.

Validation

  • make web

Notes: make web completed successfully. lit-analyse still reports existing warnings in unrelated drawer/captcha files.

Agent-thread: https://sdko.org/internal/thr/ak/019ed837-6a7a-7492-bc76-93d55a130a27

A7k-product: product

A7k-product-repo: 1

Co-authored-by: Agent <gptagent@svc.sdko.net>
@dominic-r dominic-r marked this pull request as ready for review June 18, 2026 01:00
@dominic-r dominic-r requested a review from a team as a code owner June 18, 2026 01:00
@dominic-r dominic-r self-assigned this Jun 18, 2026
@dominic-r dominic-r added the area:frontend Features or issues related to the browser, TypeScript, Node.js, etc label Jun 18, 2026
@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.28%. Comparing base (52674af) to head (6006197).
⚠️ Report is 29 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #23169      +/-   ##
==========================================
+ Coverage   93.18%   93.28%   +0.09%     
==========================================
  Files        1032     1035       +3     
  Lines       60196    60314     +118     
  Branches      400      400              
==========================================
+ Hits        56094    56263     +169     
+ Misses       4102     4051      -51     
Flag Coverage Δ
conformance 36.46% <ø> (-0.07%) ⬇️
e2e 41.70% <ø> (+0.14%) ⬆️
integration 32.90% <ø> (-0.06%) ⬇️
rust 0.00% <ø> (ø)
unit 92.22% <ø> (+0.01%) ⬆️
unit-migrate 92.25% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@netlify

netlify Bot commented Jun 18, 2026

Copy link
Copy Markdown

Deploy Preview for authentik-storybook ready!

Name Link
🔨 Latest commit 6006197
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/6a33430bb7fa8900080d9e66
😎 Deploy Preview https://deploy-preview-23169--authentik-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Jun 18, 2026

Copy link
Copy Markdown

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 6006197
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/6a33430b6cfef600076a0145
😎 Deploy Preview https://deploy-preview-23169--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Jun 18, 2026

Copy link
Copy Markdown

Deploy Preview for authentik-integrations ready!

Name Link
🔨 Latest commit 6006197
🔍 Latest deploy log https://app.netlify.com/projects/authentik-integrations/deploys/6a33430b39385c0008263e80
😎 Deploy Preview https://deploy-preview-23169--authentik-integrations.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions

Copy link
Copy Markdown
Contributor

authentik PR Installation instructions

Instructions for docker-compose

Add the following block to your .env file:

AUTHENTIK_IMAGE=ghcr.io/goauthentik/dev-server
AUTHENTIK_TAG=gh-600619721bd35e4a40499d8c22e3dfa51c374cdc
AUTHENTIK_OUTPOSTS__CONTAINER_IMAGE_BASE=ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s

Afterwards, run the upgrade commands from the latest release notes.

Instructions for Kubernetes

Add the following block to your values.yml file:

authentik:
    outposts:
        container_image_base: ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s
global:
    image:
        repository: ghcr.io/goauthentik/dev-server
        tag: gh-600619721bd35e4a40499d8c22e3dfa51c374cdc

Afterwards, run the upgrade commands from the latest release notes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:frontend Features or issues related to the browser, TypeScript, Node.js, etc

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

1 participant