Skip to content

Comments

feat: add full include query option#306

Open
mihailkitrozoskicodechem wants to merge 3 commits intomainfrom
refactor/add-full-include-query-option
Open

feat: add full include query option#306
mihailkitrozoskicodechem wants to merge 3 commits intomainfrom
refactor/add-full-include-query-option

Conversation

@mihailkitrozoskicodechem
Copy link
Collaborator

What Changed?

Add optional 'include=*' query parameter

Why?

The 'include=*' query parameter allows the user to include all the related properties

Type of Change

  • 🐛 Bug fix (fixes an issue without breaking existing functionality)
  • ✨ New feature (adds new functionality without breaking existing features)
  • 💥 Breaking change (existing functionality will be affected)
  • 📝 Documentation update
  • 🔧 Maintenance (refactoring, dependencies, configuration)

How Did You Test This?

What I tested:

Created related entities and manually tested the flow with sending 'include=*' query parameter requests.
Cases with regular, nested includes, no include parameter and faulty includes are covered.

Review Checklist

  • My code follows the project's style and conventions
  • I've reviewed my own code for obvious issues
  • I've added comments where the code might be confusing
  • I've updated relevant documentation (if needed)
  • I've tested my changes and they work as expected

Screenshots / Additional Context

image

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