Skip to content

[web] Link dashboard issues to tracker#89

Merged
sharmila-oai merged 1 commit into
mainfrom
dev/sharmila/issue-links
Jun 5, 2026
Merged

[web] Link dashboard issues to tracker#89
sharmila-oai merged 1 commit into
mainfrom
dev/sharmila/issue-links

Conversation

@sharmila-oai

@sharmila-oai sharmila-oai commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Context

Dashboard rows show tracker issue identifiers as plain text, so operators must navigate to active issues manually.

TL;DR

Link dashboard issue identifiers to their tracker-provided URLs.

Summary

  • Preserve tracker issue URLs across running, blocked, and retry snapshots
  • Expose issue URLs in the observability state API
  • Link identifiers only when the tracker URL uses http or https
  • Add a content digest to the dashboard stylesheet URL so long-lived browser caching does not hide updated link styling

Alternatives

  • Construct tracker URLs from identifiers. This would couple the dashboard to one tracker and URL format.

Test Plan

  • make -C elixir all
  • Browser preview with synthetic tracker.example issue URLs
  • Browser computed style confirms a 1px underline
  • Invalid URL scheme remains plain text

@sharmila-oai sharmila-oai added the symphony Track Symphony work for codebase label Jun 5, 2026
@sharmila-oai
sharmila-oai force-pushed the dev/sharmila/issue-links branch from b818bad to 2ab7213 Compare June 5, 2026 20:41
@sharmila-oai
sharmila-oai marked this pull request as ready for review June 5, 2026 20:50
@sharmila-oai

Copy link
Copy Markdown
Contributor Author

@codex review

Sent by Codex

@sharmila-oai
sharmila-oai force-pushed the dev/sharmila/issue-links branch from 2ab7213 to 57b7f1b Compare June 5, 2026 21:49
@sharmila-oai

Copy link
Copy Markdown
Contributor Author

@codex review

The latest commit also adds a content-hashed dashboard stylesheet URL so updated link styling is not hidden by the long-lived asset cache.

Sent by Codex

@sharmila-oai

Copy link
Copy Markdown
Contributor Author

Current-commit review completed for 57b7f1b: no actionable findings. The focused review suite passed 56 tests, and the GitHub make-all and PR-description checks are green. A redundant full local review run encountered unrelated local IPC and dependency-tooling failures; GitHub CI completed successfully on the same commit.

Sent by Codex

@sharmila-oai
sharmila-oai merged commit b4ccf7b into main Jun 5, 2026
3 checks passed
@sharmila-oai
sharmila-oai deleted the dev/sharmila/issue-links branch June 5, 2026 22:56
wildmaker pushed a commit to wildmaker/symphony that referenced this pull request Jun 8, 2026
#### Context

Dashboard rows show tracker issue identifiers as plain text, so
operators must navigate to active issues manually.

#### TL;DR

*Link dashboard issue identifiers to their tracker-provided URLs.*

#### Summary

- Preserve tracker issue URLs across running, blocked, and retry
snapshots
- Expose issue URLs in the observability state API
- Link identifiers only when the tracker URL uses `http` or `https`
- Add a content digest to the dashboard stylesheet URL so long-lived
browser caching does not hide updated link styling

#### Alternatives

- Construct tracker URLs from identifiers. This would couple the
dashboard to one tracker and URL format.

#### Test Plan

- [x] `make -C elixir all`
- [x] Browser preview with synthetic `tracker.example` issue URLs
- [x] Browser computed style confirms a `1px` underline
- [x] Invalid URL scheme remains plain text
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

symphony Track Symphony work for codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant