Skip to content

Commit

Permalink
Merge pull request #199 from intuitem/CA-314-remove-link-to-requireme…
Browse files Browse the repository at this point in the history
…nt-assessment-view-in-breadcumb

Remove the link to the requirement-assessment list view that should't exist in the breadcrumb
  • Loading branch information
monsieurswag authored Apr 5, 2024
2 parents efb4294 + 3df6a20 commit 23d0fa4
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion frontend/src/lib/components/Breadcrumbs/Breadcrumbs.svelte
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
}
return {
label: $page.data.label || t,
href: Object.keys(listViewFields).includes(tokens[0]) ? tokenPath : null
href: Object.keys(listViewFields).includes(tokens[0]) && !listViewFields[tokens[0]].breadcrumb_link_disabled ? tokenPath : null
};
});
Expand Down
3 changes: 2 additions & 1 deletion frontend/src/lib/utils/table.ts
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,8 @@ export const listViewFields = {
},
'requirement-assessments': {
head: ['name', 'description', 'complianceAssessment'],
body: ['name', 'description', 'compliance_assessment']
body: ['name', 'description', 'compliance_assessment'],
breadcrumb_link_disabled: true
},
evidences: {
head: ['name', 'file', 'description'],
Expand Down

0 comments on commit 23d0fa4

Please sign in to comment.