Skip to content

2044 bug missing color labeling of tasks#2053

Open
inadequate777 wants to merge 24 commits into
devfrom
2044-bug-missing-color-labeling-of-tasks
Open

2044 bug missing color labeling of tasks#2053
inadequate777 wants to merge 24 commits into
devfrom
2044-bug-missing-color-labeling-of-tasks

Conversation

@inadequate777
Copy link
Copy Markdown

  • added missing fields for color in taskwrapperdisplay

@inadequate777 inadequate777 requested a review from jessevz May 5, 2026 05:49
@inadequate777
Copy link
Copy Markdown
Author

need to add some test before merge

inadequate777 and others added 6 commits May 5, 2026 09:38
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
@inadequate777 inadequate777 reopened this May 5, 2026
@inadequate777
Copy link
Copy Markdown
Author

Added test for color field

Comment thread ci/apiv2/test_taskwrapperdisplays.py Outdated
Comment thread ci/apiv2/test_taskwrapperdisplay.py Outdated
@inadequate777
Copy link
Copy Markdown
Author

@inadequate777 inadequate777 requested a review from jessevz May 7, 2026 11:15
@inadequate777
Copy link
Copy Markdown
Author

  • tests work after python-hashtopolis merge

tw.taskWrapperId AS taskWrapperId, tw.priority AS taskWrapperPriority, tw.maxAgents AS taskWrapperMaxAgents,
tw.taskType AS taskType, tw.hashlistId AS hashlistId, tw.accessGroupId AS accessGroupId,
tw.taskWrapperName AS taskWrapperName, tw.isArchived AS taskWrapperIsArchived, tw.cracked AS cracked,
t.taskId AS taskId, t.taskName AS taskName, t.attackCmd AS attackCmd, t.chunkTime AS chunkTime,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You cant change a migration that has already happened, this way systems that have performed the migration will end up in a bad state. You should create a new migration where you add this field

@inadequate777 inadequate777 requested a review from jessevz May 18, 2026 13:28
@inadequate777
Copy link
Copy Markdown
Author

added new files for updating taskwrapperdisplay view using CREATE OR REPLACE VIEW in both sql and postres

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.

3 participants