sebse/factorio-signal-exporter
1
0
Fork 0
Code Issues 25 Pull Requests 6 Actions Packages Projects Releases Wiki Activity

[#5] B5: UPS chart re-sorts already-sorted data #30

Open
sebse wants to merge 1 commits from 5-remove-ups-sort into main
pull from: 5-remove-ups-sort
merge into: sebse:main
Conversation 0 Commits 1 Files Changed 1 +5 -7
sebse commented 2026-06-05 11:11:11 +00:00
Owner
Copy Link

File: web/components/ChartCard/UpsChart.tsx\n\nAPI returns UPS data ORDER BY real_time ASC. Only sort in tick mode (where game_tick order may differ from real_time). Real mode skips sort entirely.\n\nFixes #5

**File:** web/components/ChartCard/UpsChart.tsx\n\nAPI returns UPS data ORDER BY real_time ASC. Only sort in tick mode (where game_tick order may differ from real_time). Real mode skips sort entirely.\n\nFixes #5
sebse added the bug label 2026-06-05 11:11:11 +00:00
sebse added 1 commit 2026-06-05 11:11:11 +00:00
fix: skip sort in real mode, API already returns ORDER BY real_time ASC b3aec5e6b2
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin 5-remove-ups-sort:5-remove-ups-sort
git checkout 5-remove-ups-sort
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is broken
 code-smell
Code quality or maintainability issue
 enhancement
New feature or extension idea
 low-hanging-fruit
Easy to fix, high impact
No Label bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
sebse
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sebse/factorio-signal-exporter#30
Delete Branch "5-remove-ups-sort"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?