-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🐛 Exclude non-required questionnaires in assessed calculation #1666
Conversation
Signed-off-by: ibolton336 <[email protected]>
b691113
to
9e1a64b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few comments and thoughts
const assessmentQueries = useQueries({ | ||
queries: archetypes.map((archetype) => ({ | ||
queryKey: ["assessmentsForArchetype", archetype.id], | ||
queryFn: () => getAssessmentsByItemId(true, archetype.id), // Replace with actual API call |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this comment still valid?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, kinda a bummer we can't just send a single request to "get assessments for this list of archetype ids".
...p/pages/applications/components/application-detail-drawer/components/assessed-archetypes.tsx
Outdated
Show resolved
Hide resolved
...p/pages/applications/components/application-detail-drawer/components/assessed-archetypes.tsx
Outdated
Show resolved
Hide resolved
...p/pages/applications/components/application-detail-drawer/components/assessed-archetypes.tsx
Outdated
Show resolved
Hide resolved
Signed-off-by: ibolton336 <[email protected]>
Signed-off-by: ibolton336 <[email protected]>
Resolves https://issues.redhat.com/browse/MTA-1967 Addresses an issue where the assessed boolean for archetypes is set to true when all questionnaires are marked as not-required. Uses the new assessment.required boolean surfaced in konveyor/tackle2-hub@2552233 to filter out these false positives. --------- Signed-off-by: ibolton336 <[email protected]>
Resolves https://issues.redhat.com/browse/MTA-1967
Addresses an issue where the assessed boolean for archetypes is set to true when all questionnaires are marked as not-required. Uses the new assessment.required boolean surfaced in konveyor/tackle2-hub@2552233 to filter out these false positives.