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

Feature/add perf summary api - lots of improvements, not just performance summary. not bothering with bump since not published yet #1

Merged
merged 14 commits into from
Jun 12, 2024

Conversation

NodeJSmith
Copy link
Owner

No description provided.

feat(performance_api.py): add new PerformanceApi class for performance summary requests
…nfiguration

refactor(responses/*.py): inherit from OtfBaseModel
…ails

- Replace `home_studio_uuid` and `member_tz` with direct `member` and `home_studio` objects for clarity and to reduce redundant API calls
- Remove `member_home_studio` as it's now redundant with `home_studio`

refactor(member_api): change `statuses` param to `status` for bookings

- Modify `get_member_bookings` to accept a single `BookingStatus` instead of a list to match API behavior
- Update method documentation to reflect changes and clarify API behavior

feat(models): add `BookingStatus`, `PerformanceSummaryDetail`, and `PerformanceSummaryList` to models

- Introduce new models to support additional booking statuses and performance summaries
- Update `__all__` exports to include new models for package consumers

refactor(classes_api): use new `home_studio` object for studio UUID references

- Simplify studio UUID references by using the `home_studio` object directly

style(api): enhance logging for request method with params and kwargs

- Improve debug logging in `Api._default_request` to include `params` and `kwargs` for better traceability

refactor(__init__): replace `ClassStatus` with `BookingStatus` across modules

- Update import and export lists to reflect the replacement of `ClassStatus` with `BookingStatus`

feat(performance_summary_detail): create new model for detailed performance summaries

- Add `PerformanceSummaryDetail` model to represent detailed performance data for classes

The changes were made to improve code clarity, reduce redundancy, and support new features related to booking statuses and performance summaries. The refactorings ensure that the codebase is more maintainable and aligns with the actual behavior of the underlying API. The new features allow for better representation of booking and performance data within the application.

feat(models/responses): add PerformanceSummaryList model for structured responses

feat(performance_api): return PerformanceSummaryList and PerformanceSummaryDetail
refactor(performance_api): use structured models for API response data

refactor(studio_api): use home_studio_uuid from home_studio object
feat(studio_api): allow optional latitude and longitude for studio search
refactor(studio_api): default to home studio location if no geo params provided
@NodeJSmith NodeJSmith merged commit 34836ff into main Jun 12, 2024
4 checks passed
@NodeJSmith NodeJSmith deleted the feature/add_perf_summary_api branch June 12, 2024 21:52
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