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

feat(directory): Add service to cart from service details page #4540

Closed
connoratrug opened this issue Dec 4, 2024 · 0 comments · Fixed by #4543
Closed

feat(directory): Add service to cart from service details page #4540

connoratrug opened this issue Dec 4, 2024 · 0 comments · Fixed by #4543
Assignees
Labels
app:directory enhancement New feature or request

Comments

@connoratrug
Copy link
Contributor

Issue

If a user moves from the cards view to the service detail view its nice if the user can decide to add/remove the service directly form the details view ( instead of going back to the card). This is also consistent with the adding/removing of the collections

Solution

add a add/remove button on the service details page and link it to the cart

Alternatives

do nothing, services can still be added/removed just takes more clicks and is less consistent

Additional context

The directory app model was updated so the user can now request services

related to: https://github.com/orgs/molgenis/projects/15/views/1?sliceBy%5Bvalue%5D=Dev&pane=issue&itemId=71413604&issue=molgenis%7CGCC%7C555

and

#4501

@connoratrug connoratrug added enhancement New feature or request app:directory labels Dec 4, 2024
@mswertz mswertz changed the title Directory: Add service to cart from service details page feat(directory): Add service to cart from service details page Dec 5, 2024
@connoratrug connoratrug self-assigned this Dec 5, 2024
@connoratrug connoratrug linked a pull request Dec 5, 2024 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app:directory enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant