[#72855] Add semantic identifier support to "Add Project"#22330
Merged
thykel merged 18 commits intofeature/71896-change-identifier-with-semantic-identifiersfrom Mar 19, 2026
Conversation
…rs' into feature/72855-new-project-with-semantic-identifiers
…rs' into feature/72855-new-project-with-semantic-identifiers
…rs' into feature/72855-new-project-with-semantic-identifiers
This reverts commit b1fff98.
3 tasks
Contributor
Author
Deploying openproject with ⚡ PullPreview
|
akabiru
requested changes
Mar 18, 2026
Member
akabiru
left a comment
akabiru
left a comment
There was a problem hiding this comment.
Coming along nicely! A couple of remarks 🤼
frontend/src/stimulus/controllers/dynamic/projects/identifier-suggestion.controller.ts
Show resolved
Hide resolved
frontend/src/stimulus/controllers/dynamic/projects/identifier-suggestion.controller.ts
Show resolved
Hide resolved
frontend/src/stimulus/controllers/dynamic/projects/identifier-suggestion.controller.ts
Show resolved
Hide resolved
frontend/src/stimulus/controllers/dynamic/projects/identifier-suggestion.controller.ts
Show resolved
Hide resolved
akabiru
reviewed
Mar 18, 2026
|
|
||
| if (this.identifierInput) { | ||
| this.identifierInput.readOnly = true; | ||
| this.identifierInput.placeholder = I18n.t('js.projects.identifier_suggestion.loading'); |
Member
There was a problem hiding this comment.
🍊 For loading state- you might want to consider: https://primer.style/product/components/text-input/#in-a-loading-state due to a11ly
…rs' into feature/72855-new-project-with-semantic-identifiers
thykel
merged commit dd804dc
into
feature/71896-change-identifier-with-semantic-identifiers
9 of 11 checks passed
Contributor
Author
|
Huh, I thought I somehow understand git, but I'm not sure what happened here. Running Likely something I messed up in my own repository copy? I'm just gonna port the PR feedback commits into the older PR, then we can split it again if needed. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ticket
https://community.openproject.org/projects/stream-jira-exit/work_packages/72855
What are you trying to accomplish?
We are currently adding support for "semantic work package identifiers".
project-zeus/ work package ID1PZEUS/ work packagePZEUS-1.This PR extends the "New Project" page to include an identifier form that gets auto-filled with a suggestion when a user types a project name and then clicks or tabs outside the name input form.
The identifier suggestions are pulled from the API and support both the legacy format as well as the new semantic format.
Screenshots
What approach did you choose and why?
Merge checklist