-
Notifications
You must be signed in to change notification settings - Fork 433
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
Added missing/incomplete breadcrumbs on create community/collection/item pages #2888
Added missing/incomplete breadcrumbs on create community/collection/item pages #2888
Conversation
Hi @alexandrevryghem, |
…ssing-breadcrumbs_contribute-main # Conflicts: # src/app/collection-page/collection-page-routes.ts # src/app/community-page/community-page-routes.ts # src/app/core/breadcrumbs/community-breadcrumb.resolver.ts # src/app/core/breadcrumbs/dso-breadcrumb.resolver.spec.ts # src/app/core/breadcrumbs/dso-breadcrumb.resolver.ts # src/app/core/submission/resolver/submission-object.resolver.ts # src/app/workflowitems-edit-page/item-from-workflow.resolver.spec.ts # src/app/workflowitems-edit-page/item-from-workflow.resolver.ts # src/app/workflowitems-edit-page/workflowitems-edit-page-routes.ts # src/app/workspaceitems-edit-page/item-from-workspace.resolver.spec.ts # src/app/workspaceitems-edit-page/item-from-workspace.resolver.ts # src/app/workspaceitems-edit-page/workspaceitems-edit-page-routes.ts
…to w2p-112970_added-missing-breadcrumbs_contribute-main # Conflicts: # src/app/collection-page/collection-page-routing.module.ts # src/app/community-page/community-page-routing.module.ts
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.
👍 Thanks @alexandrevryghem ! Verified this provides a much better experience. Breadcrumbs now shown in submission/workflow, and on all create forms. Everywhere else they work as normal. I've added the port to dspace-7_x
label to this PR as this would likely be good to port to 7.x as well, if possible to do so.
Backport failed for Please cherry-pick the changes locally and resolve any conflicts. git fetch origin dspace-7_x
git worktree add -d .worktree/backport-2888-to-dspace-7_x origin/dspace-7_x
cd .worktree/backport-2888-to-dspace-7_x
git switch --create backport-2888-to-dspace-7_x
git cherry-pick -x 692bb991a051c9491769ba10db1425f1f9573849 60d93e653fac08dbaac87c0a5efa19581d259386 4251630ab86d1430f1cb45116097fa7d30d7d8d2 |
As I expected, this wasn't able to port automatically to 7.x. So, this would need a manual port PR, if we chose to port it. If it's not easily possible, please let me know and we can simply leave this in 8.0 only. |
References
Description
Currently the create community & collection pages don't have any breadcrumbs this PR fixes this. The workspace item/workflow item forms also don't contain the whole parent comcol structure. Having this structure there too can be handy to be able to quickly check that you selected the correct collection. Currently the URLs like
/submit?collection={uuid}
still don't have breadcrumbs because they are almost instantly redirected to/workspaceitems/{id}/edit
, but this might still be handy to have this for slower serversInstructions for Reviewers
List of changes in this PR:
CommunityBreadcrumbResolver
to use the parameter name provided through the route data in thebreadcrumbQueryParam
property, to find the community/sub-community structure to which the sub-community/collection belongs.I18nBreadcrumbResolver
on the create community/collection route to display the textCreate community
/Create collection
SubmissionParentBreadcrumbResolver
containing the logic of the newItemFromWorkspaceBreadcrumbResolver
&ItemFromWorkflowBreadcrumbResolver
. These are the resolvers that will create the whole comcol parent structure based on the collection of theWorkspaceItem
/WorkflowItem
. This resolver also listens toChangeSubmissionCollectionAction
store dispatches in order to recreate the breadcrumbs when you change the collection of the workspace item.Include guidance for how to test or review your PR.
Checklist
yarn lint
yarn check-circ-deps
)package.json
), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.