Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Change details in task class #664

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

Merged
merged 4 commits into from
Jan 30, 2023

Conversation

Azanul
Copy link
Contributor

@Azanul Azanul commented Jan 24, 2023

Pull Request

Related issue

Fixes #602

PR checklist

Please check if your PR fulfills the following requirements:

  • Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
  • Have you read the contributing guidelines?
  • Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

@Azanul Azanul marked this pull request as ready for review January 24, 2023 11:48
Copy link
Contributor

@alallema alallema left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Azanul,
Thanks for this PR! I left a comment

Copy link
Contributor

@alallema alallema left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! πŸŽ‰

@alallema
Copy link
Contributor

bors merge

bors bot added a commit that referenced this pull request Jan 30, 2023
664: Change details in task class r=alallema a=Azanul

# Pull Request

## Related issue
Fixes #602

## PR checklist
Please check if your PR fulfills the following requirements:
- [x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
- [x] Have you read the contributing guidelines?
- [x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!


Co-authored-by: Azanul <[email protected]>
Co-authored-by: Azanul Haque <[email protected]>
@bors
Copy link
Contributor

bors bot commented Jan 30, 2023

Build failed:

@alallema
Copy link
Contributor

bors merge

@bors
Copy link
Contributor

bors bot commented Jan 30, 2023

@bors bors bot merged commit 0b509dd into meilisearch:main Jan 30, 2023
@alallema alallema mentioned this pull request Jan 30, 2023
3 tasks
@alallema alallema added the breaking-change The related changes are breaking for the users label Feb 6, 2023
bors bot added a commit that referenced this pull request Feb 6, 2023
678: Update version for the next release (v0.24.0) r=alallema a=alallema

This version makes this package compatible with Meilisearch v1.0.0 πŸŽ‰
Check out the changelog of [Meilisearch v1.0.0](https://github.com/meilisearch/meilisearch/releases/tag/v1.0.0) for more information on the changes(#635).

## ⚠️ Breaking changes

* Make the usage of the module typing consistent (#625) `@pbrochar`
* Put all dates to `datetime` type for consistency. Class impacted: `Task` and `TasInfo` (#661) `@Azanul`
* Change `details` fields in `Task` class `Dict[str, Any]` -> `Union[Dict[str, Any], None]` (#664) `@Azanul`

## πŸš€ Enhancements

* Added `update_documents_csv(str_documents: str, primary_key: Optional[str] = None) -> TaskInfo` (#654) `@sajdakabir`

## πŸ› Bug Fixes

* Include py.typed in package build (#637) `@sanders41`

Thanks again to `@Azanul,` `@alallema,` `@pbrochar,` `@sajdakabir,` `@sanders41!` πŸŽ‰



Co-authored-by: alallema <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change The related changes are breaking for the users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change type for details in Task class
3 participants