Skip to content
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

A recent change to tackle2-hub breaks archetype functionality in UI #1436

Closed
1 task done
sjd78 opened this issue Oct 5, 2023 · 0 comments · Fixed by #1438
Closed
1 task done

A recent change to tackle2-hub breaks archetype functionality in UI #1436

sjd78 opened this issue Oct 5, 2023 · 0 comments · Fixed by #1438
Assignees
Labels
Custom Assessment Items relating to custom assessment work kind/bug Categorizes issue or PR as related to a bug.
Milestone

Comments

@sjd78
Copy link
Member

sjd78 commented Oct 5, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Konveyor version

main

Priority

Critical

Current Behavior

A new Archetype cannot be created. Error shown is json: unknown field "criteriaTags".

In the archetype details drawer, an archetype's assessment tags are not displayed.

An archetype or application cannot be assessed. Fixed on hub: konveyor/tackle2-hub#506

Expected Behavior

Archetype creation should work.

Assessing an archetype should work.

The tags from an archetype's assessments should be displayed in the detail drawer.

How Reproducible

Always (Default)

Steps To Reproduce

No response

Environment

No response

Anything else?

The tackle2-hub pull request with the rest api change: konveyor/tackle2-hub#502

@sjd78 sjd78 added kind/bug Categorizes issue or PR as related to a bug. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels Oct 5, 2023
@sjd78 sjd78 self-assigned this Oct 5, 2023
@sjd78 sjd78 removed the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Oct 5, 2023
@sjd78 sjd78 added this to the 0.3-beta.1 milestone Oct 5, 2023
@sjd78 sjd78 added this to Konveyor UI Oct 5, 2023
@sjd78 sjd78 moved this to In Progress in Konveyor UI Oct 5, 2023
@sjd78 sjd78 added the Custom Assessment Items relating to custom assessment work label Oct 5, 2023
@sjd78 sjd78 changed the title A recent change to tackle2-hub breaks archetype creation in UI A recent change to tackle2-hub breaks archetype functionality in UI Oct 5, 2023
ibolton336 pushed a commit that referenced this issue Oct 6, 2023
…1438)

### Summary

Resolves: #1436

With hub updates to tag handling and api field name changes related to
archetypes, adjustments needed to be made to the UI.

### Archetype model
- Update `models.ts` to the current state of `TagRef` and `Archetype` in
terms of field names and data types.

  - MSW stub data updated to use api model changes.

### Archetype Table 
- **Note**: The tags column on the archetype table is currently
displaying ALL tags returned with no source differentiation.

### Archetype Add/EditForm
- Update `ArchetypeForm` to use `criteria` instead of `criteriaTags` as
per the api model changes.

- With hub changes to combine the archetype and non-archetype tags into
a single field, the form processing now ignored non-archetype fields but
still sends them on an update.

- Note: This change mirrors the behavior of manual / non-manual tags on
applications and the application form.

- Stakeholder and stakeholder groups handling is updated to use the
`*ToRef()` helper function style introduced in the application form
change #1408.

### Archetype Detail Drawer

- Since all tags for an archetype are provided in the same field `tags`,
filter them into "archetype tags" (i.e. an empty source) and "assessment
tags" (i.e. have a non-empty source) to display them separately.

- Note: The "archetype tags" are the tags manually attached to the
archetype in the new/edit form.

---------

Signed-off-by: Scott J Dickerson <sdickers@redhat.com>
@github-project-automation github-project-automation bot moved this from In Progress to Done in Konveyor UI Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Custom Assessment Items relating to custom assessment work kind/bug Categorizes issue or PR as related to a bug.
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant