servala/servala-portal

Fork 0

WIP: Service instance detail page improvements #333

Draft

rixx wants to merge 6 commits from 257-service-detail-page into main

rixx commented

2025-12-09 14:53:55 +00:00

Member

#257

Not tested yet due to the k8s cluster being down

#257 **Not tested yet due to the k8s cluster being down**

rixx added 5 commits 2025-12-09 14:53:56 +00:00

Show times as local time 15b5d78c62

Toggle credentials in instance detail view feede16265

fixup! Show times as local time c36ee2494d

Prettier rendering for JSON data on instance page ed0ff0f380

Show k8s event on instance page

Tests / test (push) Successful in 27s

Details

63b451b234

rixx added 1 commit 2025-12-09 15:24:36 +00:00

Move events to end of page and take up full width

Tests / test (push) Successful in 28s

Details

5e543b1e4c

tobru changed title from ~~Service instance detail page improvements~~ to WIP: Service instance detail page improvements

2025-12-11 10:29:00 +00:00

tobru referenced this pull request

2025-12-11 15:27:54 +00:00

redesign service detail page #334

rixx added 2 commits 2025-12-12 07:47:07 +00:00

redesign service detail page

Tests / test (push) Successful in 56s

Details

5c39d26b62

with the help of Claude Code

Merge pull request 'redesign service detail page' (#334 )

Tests / test (push) Successful in 30s

Details

275e5e3cc1

Reviewed-on: #334
Reviewed-by: Tobias Kunze <r@rixx.de>

rixx force-pushed 257-service-detail-page from 275e5e3cc1 to 1ac799e29c

2025-12-12 07:51:52 +00:00

Compare

rixx commented

2025-12-12 07:52:17 +00:00

Author

Member

Open questions:

Where on the Claude-designed page should the instance name go (as opposed to the display name)? (@tobru )

TODO:

Refactor duplicated code in service instance detail view

Open questions: - Where on the Claude-designed page should the instance name go (as opposed to the display name)? (@tobru ) TODO: - Refactor duplicated code in service instance detail view

rixx force-pushed 257-service-detail-page from 1ac799e29c to 4ae27e4109

2025-12-12 07:52:35 +00:00

Compare

tobru commented

2025-12-12 14:07:06 +00:00

Owner

Where on the Claude-designed page should the instance name go (as opposed to the display name)?

In the title as well:

Instance Display Name (Instance ID)

The instance ID should be formatted as a mono font.

> Where on the Claude-designed page should the instance name go (as opposed to the display name)? In the title as well: `Instance Display Name (Instance ID)` The instance ID should be formatted as a mono font.

This pull request is marked as a work in progress.

This branch is out-of-date with the base branch

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin 257-service-detail-page:257-service-detail-page

git checkout 257-service-detail-page

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout main

git merge --no-ff 257-service-detail-page

git checkout 257-service-detail-page

git rebase main

git checkout main

git merge --ff-only 257-service-detail-page

git checkout 257-service-detail-page

git rebase main

git checkout main

git merge --no-ff 257-service-detail-page

git checkout main

git merge --squash 257-service-detail-page