Skip to content

docs: migrate to v2 temporal-ui#952

Open
asanvaq wants to merge 8 commits into
canonical:mainfrom
asanvaq:temporal-ui
Open

docs: migrate to v2 temporal-ui#952
asanvaq wants to merge 8 commits into
canonical:mainfrom
asanvaq:temporal-ui

Conversation

@asanvaq
Copy link
Copy Markdown
Contributor

@asanvaq asanvaq commented Apr 18, 2026


Description

Update docs to v2.

@zhijie-yang zhijie-yang added the documentation Improvements or additions to documentation label Apr 20, 2026
Comment thread oci/temporal-ui/documentation.yaml Outdated
Comment thread oci/temporal-ui/documentation.yaml Outdated
Comment thread oci/temporal-ui/documentation.yaml
Comment thread oci/temporal-ui/documentation.yaml Outdated
Comment thread oci/temporal-ui/documentation.yaml Outdated
Comment thread oci/temporal-ui/documentation.yaml Outdated
Comment thread oci/temporal-ui/documentation.yaml Outdated
Comment thread oci/temporal-ui/documentation.yaml Outdated
Comment thread oci/temporal-ui/documentation.yaml Outdated
Comment thread oci/temporal-ui/documentation.yaml Outdated
asanvaq and others added 2 commits May 5, 2026 10:25
@asanvaq
Copy link
Copy Markdown
Contributor Author

asanvaq commented May 5, 2026

The last two options in the config are not included in the v1 parameters, but they are included in the run example. In v2, should they go under config:, as is the case in this PR or under docker.parameters?

Copy link
Copy Markdown
Collaborator

@zhijie-yang zhijie-yang left a comment

Choose a reason for hiding this comment

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

config part looks good. some modifications to the description/docker part is still necessary.

Comment thread oci/temporal-ui/documentation.yaml Outdated
Comment thread oci/temporal-ui/documentation.yaml Outdated
@asanvaq asanvaq requested a review from zhijie-yang May 21, 2026 18:46
parameters: >-
-p 8081:8081
run_cmd: >-
-d -v <path>:/home/ui-server/config -e TEMPORAL_ROOT=/home/ui-server -e TEMPORAL_ENVIRONMENT=production
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.

They should all go to parameters instead of run_cmd.

The temporal UI service is started by Pebble and serves the UI on port 8081.
The Temporal UI is started by Pebble and serves the UI on port 8081.

To obtain a runnable configuration, clone https://github.com/temporalio/ui.
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.

Not a blocker, but this reads odd. I'd use To obtain a runnable configuration, refer to the [official repository](https://github.com/temporalio/ui).

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

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants