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.

Sign up for GitHub

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

Jump to bottom

Refactor MistralClient Class #5

Open
markjamesm opened this issue Dec 16, 2023 · 1 comment
Open

Refactor MistralClient Class #5

markjamesm opened this issue Dec 16, 2023 · 1 comment
Assignees
@markjamesm
Labels
enhancement New feature or request
Milestone
1.0

Comments

@markjamesm
Copy link
Owner

I've been working rather quickly to implement the API, and a refactor is needed to cleanup the MistralClient class. I plan to split the GetAvailableModelsAsync() and ChatAsync() methods into smaller private methods to handle the conversion to DTO classes as they're currently quite long.
@markjamesm markjamesm added the enhancement New feature or request label Dec 16, 2023
@markjamesm markjamesm added this to the 1.0 milestone Dec 16, 2023
@markjamesm markjamesm self-assigned this Dec 16, 2023
@markjamesm
Copy link
Owner Author

Currently working on this, started with this commit:

4b6c03e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@markjamesm markjamesm

Labels
enhancement New feature or request
Projects
None yet
Milestone
1.0
Development

No branches or pull requests
1 participant
@markjamesm