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: add function logs API [EXT-6001] #2520

Merged
merged 15 commits into from
Jan 28, 2025

Conversation

michaelpineirocontentful
Copy link
Contributor

Summary

Add function log endpoints to be used through plain client

  • Added function log entity and plain client
  • Added unit tests for function log
  • Added js docstrings

Motivation and Context

We want to have access to basic FunctionLog API's through CMA.

Checklist (check all before merging)

  • Both unit and integration tests are passing
  • There are no breaking changes
  • Changes are reflected in the documentation

When adding a new method:

- Added function log entity and plain client
- Added unit tests for function log
- Added js docstrings
@michaelpineirocontentful michaelpineirocontentful marked this pull request as ready for review January 23, 2025 16:46
@michaelpineirocontentful michaelpineirocontentful requested a review from a team as a code owner January 23, 2025 16:46
test/unit/create-function-log-api.ts Outdated Show resolved Hide resolved
test/unit/create-function-log-api.ts Outdated Show resolved Hide resolved
lib/plain/entities/function-log.ts Outdated Show resolved Hide resolved
@michaelpineirocontentful michaelpineirocontentful enabled auto-merge (squash) January 28, 2025 16:56
@michaelpineirocontentful michaelpineirocontentful merged commit a9dc615 into master Jan 28, 2025
7 checks passed
@michaelpineirocontentful michaelpineirocontentful deleted the feat-add-function-logs-api-ext-6001 branch January 28, 2025 17:05
@contentful-automation
Copy link
Contributor

🎉 This PR is included in version 11.45.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants