Skip to content

feat(registry-overview): establish routing, sidenav, store #135

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

kpetrov24exoft
Copy link
Collaborator

@kpetrov24exoft kpetrov24exoft commented Jun 25, 2025

commit: [establish routing, sidenav, store]

  • establish sidenav, so Registry details is visible when registries/my-registrations/:id, in the route
    Знімок екрана 2025-06-25 134856
  • add link to registration overview page, on search page;
  • add files for store;
  • establish registry overview component;

commit: [mapper, get bookmarks, overview-toolbar]

  • overview toolbar was done only for projects, so here it's updated to be reusable for other resources;
  • also on my projects we had getBookmarks, which got only bookmarked projects, but it should get also registrations and preprints so it's got fixed to;
  • and initial mapping to get registry by id;

@kpetrov24exoft kpetrov24exoft requested a review from nsemets June 25, 2025 10:51
@kpetrov24exoft kpetrov24exoft self-assigned this Jun 25, 2025
@kpetrov24exoft kpetrov24exoft marked this pull request as draft June 25, 2025 10:55
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add to all models from json api suffix JsonApi. Also separate JsonApi models and domain models.

@kpetrov24exoft kpetrov24exoft force-pushed the feat/215-registries-overview branch from f68b7bb to 5eeac02 Compare June 30, 2025 22:58
@kpetrov24exoft kpetrov24exoft marked this pull request as ready for review June 30, 2025 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants