Skip to content

fix(customer-priority-bump): apply 'priority' label not the field (D5, unblocks Priority-field deletion) - #84

Merged
LukasWodka merged 1 commit into
developfrom
fix/priority-label-not-field
Jul 26, 2026
Merged

fix(customer-priority-bump): apply 'priority' label not the field (D5, unblocks Priority-field deletion)#84
LukasWodka merged 1 commit into
developfrom
fix/priority-label-not-field

Conversation

@LukasWodka

@LukasWodka LukasWodka commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

D5: priority is a binary label now, not a project field. This reusable still set the Priority field — which would break once the field is deleted (I've deleted 8 of the 9 dead fields; Priority is the one held back on this).

Rewrites it to gh issue edit --add-label priority. Merge → promote to main → then I delete the Priority field.


Note

Low Risk
CI-only change to how customer issues are marked; no app runtime impact, though callers expecting P1 on the project board will now only get the label until they update filters/process.

Overview
D5: Customer-flagged issues are now escalated via a priority issue label instead of setting the org project’s Priority single-select field.

The reusable customer-priority-bump workflow drops the GraphQL project lookup, retry loop for project items, and updateProjectV2ItemFieldValue mutation. On from:customer, it runs gh issue edit --add-label priority using the existing PROJECTS_KANBAN_TOKEN. Workflow comments and step naming reflect the new behavior.

Callers can still pass project-number, org, and target-priority; those inputs are no longer used by the job (only trigger-label behavior remains).

Reviewed by Cursor Bugbot for commit e1854f1. Bugbot is set up for automated code reviews on this repo. Configure here.

…d (D5)

D5 replaced the Priority single-select field with a binary 'priority' label.
This reusable still set the (now-removed) Priority field, so it would fail once
the field is deleted. Rewrite it to 'gh issue edit --add-label priority' — the
label is visible on the issue and filterable everywhere.

Prerequisite for deleting the Priority project field: this must be on main
first, or the workflow errors on the missing field.
@LukasWodka LukasWodka self-assigned this Jul 26, 2026
@LukasWodka

Copy link
Copy Markdown
Contributor Author

bugbot run

@LukasWodka

Copy link
Copy Markdown
Contributor Author

👋 Heads-up — Code review queue is at 39 / 30

Above the WIP limit. The team convention is to review existing PRs before opening new work.

Open PRs currently in Code review (oldest first):

Pull from review before opening new work. (This is a nudge from the kanban WIP check, not a block.)

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit e1854f1. Configure here.

@LukasWodka
LukasWodka merged commit e3f9d06 into develop Jul 26, 2026
5 checks passed
LukasWodka added a commit that referenced this pull request Jul 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant