Skip to content

Conversation

sinejespersen
Copy link
Contributor

@sinejespersen sinejespersen commented Aug 29, 2025

Link to LEANTIME ticket

https://leantime.itkdev.dk/#/tickets/showTicket/5316

Link to Github issue

#267

Description

  • Create helper.js for getInputFiles
  • Create renderAdminForm like already existing renderSlide
  • Rename templates.js to templates-slide.js (as the new admin renderer is called templates-admin.js)
  • Remove adminForm in image-text.json
  • Create admin-form.jsx with the admin form in, add a bit of if/then in the jsx.
  • Extract image-text template into file to simplify image-text.jsx

More info on the grand idea, see github issue

Screenshot of the result

🖼️

@sinejespersen sinejespersen marked this pull request as draft August 29, 2025 11:09
@sinejespersen sinejespersen self-assigned this Aug 29, 2025
@sinejespersen sinejespersen added the enhancement New feature or request label Aug 29, 2025
@sinejespersen sinejespersen linked an issue Aug 29, 2025 that may be closed by this pull request
7 tasks
@sinejespersen sinejespersen requested a review from tuj August 29, 2025 11:10
@@ -0,0 +1,70 @@
// Load templates.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any reason for splitting into template-admin template-slide ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dynamic admin forms
2 participants