Skip to content

[PoC][COMMS-1073] Make test suite pass under semantic identifier mode - #26567

Draft
thykel wants to merge 12 commits into
release/18.0from
feature/comms-1073-use-semantic-identifiers-by-default
Draft

thykel wants to merge 12 commits into
release/18.0from
feature/comms-1073-use-semantic-identifiers-by-default

Conversation

@thykel

@thykel thykel commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Ticket

What are you trying to accomplish?

Screenshots

What approach did you choose and why?

AI involvement

Merge checklist

  • Added/updated tests
  • Added/updated documentation in Lookbook (patterns, previews, etc)
  • Tested major browsers (Chrome, Firefox, Edge, ...)

@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown
1 Warning
⚠️ This PR has migration-related changes on a release branch. Ping @opf/operations

Generated by 🚫 Danger

thykel and others added 11 commits November 2, 2026 22:40
- Drop hardcoded classic project identifiers from feature specs; use
  project.identifier or look up UI-created projects by name.
- Use display_id/formatted_id instead of the numeric id in labels,
  links and paths, including shared page objects.
- Find the mail handler target project regardless of identifier case.
- Treat a matching displayId as a direct hit in the global search.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Select table rows by clicking above the id link instead of following it.
- Use the work package record or display id for onboarding, deep link
  and id filter expectations.
- Ignore the work package identifier when comparing template copies.
- Generate a valid project identifier for the attribute activity spec.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Select the split view's row once the table results resolve a semantic id.
- Match a newly created work package against numeric and semantic URLs.
- Keep the query title from overriding an open split view's title.
- Restore the work package title after Turbo renders the page snapshot.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
OpTitleService#setFirstPart gives a different result when called again,
and a semantic route param delivers the work package from both its
semantic and numeric cache slot.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The board specs used the project identifier "boards" to cover #29754, which
only classic identifiers allow.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch has not been deployed

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant