Skip to content

fix: reverse ordering of calls - #577

Open
zeynepcaliskan wants to merge 1 commit into
Eyevinn:mainfrom
EyevinnStudentDev:call-order-change
Open

zeynepcaliskan wants to merge 1 commit into
Eyevinn:mainfrom
EyevinnStudentDev:call-order-change

Conversation

@zeynepcaliskan

Copy link
Copy Markdown
Contributor

Update the calls UI and keyboard index mapping so that newly created calls appear at the top of the list and are assigned the lowest numeric shortcut (1, 2, 3, …). This is done by reversing the iteration order of calls both when building callIndexMap and when rendering ProductionLines.

@birme

birme commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Reviewed as part of a backlog housekeeping pass. The change itself is small and self-contained (reversing call display order in calls-page.tsx and production-lines.tsx), and the checks that ran are green — but the branch is currently conflicting with main (mergeStateStatus: DIRTY), so it can't be merged as-is. Could you rebase onto the latest main and resolve the conflict? Once it's mergeable we can complete the review and merge. Flagging for author attention.

@birme

birme commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Flagged by daily-backlog-pr Phase 3.5 housekeeping: this PR has merge conflicts with the base branch (mergeable=CONFLICTING). It needs a rebase / conflict resolution from the author before it can be reviewed for merge. Not auto-handling since resolving conflicts requires author intent.

@birme

birme commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Automated sweep (daily-backlog-pr): this PR is currently CONFLICTING with the base branch and can't be auto-merged. It needs a rebase onto the default branch and conflict resolution by the author/maintainer before review can complete. Flagging for human attention — not attempting an automated rebase of a feature branch.

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.

2 participants