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.

Sign up for GitHub

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

Jump to bottom

feat(customers): Add index and show on API #434

Merged
merged 2 commits into from
Sep 5, 2022
Merged

feat(customers): Add index and show on API #434

merged 2 commits into from
Sep 5, 2022

Conversation

jdenquin
Copy link
Contributor

@jdenquin jdenquin commented Sep 5, 2022

Context

We do not have any endpoints to list organization's customer or search for a customer on the API.

Description

  • Add the /customers index endpoint
  • Add the /customers/:customer_id show endpoint

@jdenquin jdenquin added enhancement New feature or request API Related to the public API Customer Customer management labels Sep 5, 2022
@jdenquin jdenquin self-assigned this Sep 5, 2022
rsempe
rsempe reviewed Sep 5, 2022
View reviewed changes
config/routes.rb Outdated Show resolved Hide resolved
@jdenquin jdenquin force-pushed the add-customers-listing branch from ca51e32 to 3f2d169 Compare September 5, 2022 15:26
@jdenquin jdenquin merged commit 7152b14 into main Sep 5, 2022
@jdenquin jdenquin deleted the add-customers-listing branch September 5, 2022 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rsempe rsempe rsempe approved these changes

@vincent-pochet vincent-pochet Awaiting requested review from vincent-pochet

@lovrocolic lovrocolic Awaiting requested review from lovrocolic

Assignees

@jdenquin jdenquin

Labels
API Related to the public API Customer Customer management enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jdenquin @rsempe