Skip to content

Conversation

@aliakseitokarev
Copy link
Contributor

@aliakseitokarev aliakseitokarev commented Sep 3, 2025

Pull Request Guidelines

Issue:

Description:

image image

Self-Check:

  • Database migration added (if required)
  • Changes tested locally

@aliakseitokarev aliakseitokarev changed the title feat: add button 'Done Task' for auto-tasks feat: add button 'Done Task' for auto-task Sep 3, 2025

const score = verifications?.[0]?.score ?? null;
const isMinimumScoreDone = score >= publicAttributes.tresholdPercentage;
console.log(isMinimumScoreDone);
Copy link
Contributor

Choose a reason for hiding this comment

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

please remove this

</Paragraph>
</Col>
<Col span={24}>
<Col span={24} style={{ display: 'flex', justifyContent: 'space-between' }}>
Copy link
Contributor

Choose a reason for hiding this comment

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

Will Col + Row work instead?

const [activeTab, setActiveTab] = useState(CourseTaskStatus.Available);
const statuses = useMemo(() => tasks?.map(t => t.status) || [], [tasks]);
const filteredTasks = useMemo(() => tasks?.filter(t => t.status === activeTab) || [], [tasks, activeTab]);
const isAvailableTab = activeTab === CourseTaskStatus.Available ? false : true;
Copy link
Contributor

Choose a reason for hiding this comment

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

this naming is confusing, isAvailableTab = true when active tab != "Available"

@aliakseitokarev aliakseitokarev self-assigned this Nov 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants