Skip to content

feat: add assistant model#291

Draft
whplh wants to merge 18 commits into
developmentfrom
feature/dev-branch-base-add-agents
Draft

feat: add assistant model#291
whplh wants to merge 18 commits into
developmentfrom
feature/dev-branch-base-add-agents

Conversation

@whplh
Copy link
Copy Markdown
Contributor

@whplh whplh commented May 4, 2026

Some model relations are going to change for version 2.5.0 (-> this pr is a draft)

  • The api supports listing assistants and includes fields for relational fields only when the request parameters includes them.
  • The api supports updating an existing assistant
  • The api supports deleting an assistant

See tests for request examples.


Refs: HKI-81

@whplh whplh requested a review from Ariansdf May 4, 2026 13:11
@whplh
Copy link
Copy Markdown
Contributor Author

whplh commented May 8, 2026

@Ariansdf I updated the api to support json api,
which is supported by laravel 13 out of the
box.
For filtering the recommended Query bulider lib
was added. ( E.g. filtering for category like: /api/assistants?filter[category]=general)

revisit api. Add pagination and filter.

Revisit review process
@whplh whplh force-pushed the feature/dev-branch-base-add-agents branch from 974499d to 8ba0af8 Compare May 8, 2026 09:45
@whplh whplh force-pushed the feature/dev-branch-base-add-agents branch from 9ca127f to 4474496 Compare May 13, 2026 15:11
@whplh
Copy link
Copy Markdown
Contributor Author

whplh commented May 13, 2026

I discovered the resource hooks from the laraveljsonapi package and refactored the code. With them the CRUD operations can pass through the laravelapi package/framework and let it do its thing and react to changes before or afterward.

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