Skip to content

Adding support for showing payload size, Plugin browser, custom fields editor and hot reloading plugins in web UI #77

Closed
sd416 wants to merge 0 commit into
usetrmnl:mainfrom
sd416:main
Closed

Adding support for showing payload size, Plugin browser, custom fields editor and hot reloading plugins in web UI #77
sd416 wants to merge 0 commit into
usetrmnl:mainfrom
sd416:main

Conversation

@sd416

@sd416 sd416 commented Jan 7, 2026

Copy link
Copy Markdown
Contributor

Fix for #67

  1. Add support for showing response size
  2. Add `trmnlp list` command to list plugins from TRMNL server
  3. Add plugin selector dropdown in web UI for browsing and loading plugins
  4. Add custom fields editor with support for text inputs and dropdowns
  5. Add hot-load API endpoint (/api/pull/:id) to switch plugins without restarting.
  6. Auto-populate custom field definitions from settings.yml
  7. Add Docker interactive mode documentation to README
image
compressed.mp4

@sd416 sd416 changed the title adding support for showing payload size Adding support for showing payload size, Plugin browser, custom fields editor and hot reloading plugins in web UI Jan 7, 2026
@sd416

sd416 commented Jan 24, 2026

Copy link
Copy Markdown
Contributor Author

@schrockwell, Can you please take a look at this?

@schrockwell schrockwell left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for taking the time to build out these improvements. There are some great ideas in here, and also some that are out of scope.

Additionally, this PR attempts to merge too many changes at once. It makes it difficult to review and discuss individual features. So I ask that you please split it out into a few separate branches:

  1. Fix #67 (see note below)
  2. Add trmnlp list
  3. Add custom field editor
  4. Add to README

Acceptable

Add trmnlp list command to list plugins from TRMNL server

Great addition! Please see the review comments for a few tweaks, and split out to a new PR.

Add custom fields editor with support for text inputs and dropdowns
Auto-populate custom field definitions from settings.yml

This feature is really nice. I am inclined to approve it, but please split into a separate PR.

Add Docker interactive mode documentation to README

This is a great tidbit, thank you.

Won't Accept

Add support for showing response size

This actually does not solve #67. The goal is to show the size of the data payload (JSON, XML, CSV, text, etc) from polling or webhook. On the positive side, the actual fix is much simpler.

Add plugin selector dropdown in web UI for browsing and loading plugins
Add hot-load API endpoint (/api/pull/:id) to switch plugins without restarting.

Sorry, these changes cannot be accepted as currently implemented. It goes against the design philosophy of trmnlp which is to serve up a single plugin from a directory. Overwriting the src/ directory is potentially destructive and makes very little sense for directories that are under version control.

Comment thread lib/trmnlp/app.rb Outdated

begin
api = APIClient.new(@context.config)
response = api.get_plugin_settings

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

  1. This should return only plugin_settings with plugin_id == 37 which is the ID for private plugins.
  2. Please sort the response, preferably by name.

@sd416

sd416 commented Feb 10, 2026

Copy link
Copy Markdown
Contributor Author

#84 created for trmnlp list support

@sd416

sd416 commented Feb 10, 2026

Copy link
Copy Markdown
Contributor Author

#85 is created for docker interactive readme update

@sd416

sd416 commented Feb 10, 2026

Copy link
Copy Markdown
Contributor Author

#86 is created for adding support for field editor on the webui

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants