Skip to content

feat: hide services from top-level - #10

Merged
coladarci merged 1 commit into
mainfrom
feat/hide-services
Jun 30, 2026
Merged

feat: hide services from top-level#10
coladarci merged 1 commit into
mainfrom
feat/hide-services

Conversation

@coladarci

@coladarci coladarci commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds getAllActivities() and getAllAddons() to ProductService — each makes only the one relevant network request instead of fetching both product types
  • Exposes every public service method as a top-level short-form on PeekAccessService (e.g. peek.getAllActivities(), peek.getBookingById(), peek.cancelBooking()); all are backwards-compatible — existing get<Resource>Service().<method>() patterns unchanged
  • Resource noun prefix used where names would collide (getTimeslotById vs getBookingById, etc.)

Test plan

  • peek.getAllActivities() returns activity products only (no add-ons)
  • peek.getAllAddons() returns add-on products only
  • All existing get<Resource>Service().<method>() call patterns still work unchanged
  • 607 tests pass, 100% function coverage, all thresholds above 95%

@coladarci
coladarci force-pushed the feat/hide-services branch from 8d466b9 to 6b82960 Compare June 30, 2026 19:56
@coladarci
coladarci merged commit 5eeaad4 into main Jun 30, 2026
2 checks passed
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.

1 participant