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

Fix deserialize operations objects #198

Merged

Conversation

sfmok
Copy link
Contributor

@sfmok sfmok commented Nov 20, 2024

This PR is a fix for #197

Before fix
img2

After fix
img3

@damienalexandre
Copy link
Member

Looks great thanks!

Can you add a note in the CHANGELOG?

A new "Fixed" section here please:

elastically/CHANGELOG.md

Lines 10 to 22 in 1bbd506

## 2.0.0
Massive release with added support for Elastica 8 and all deprecated code removed.
### Added
- Upgraded `ruflin/elastica` to v8 (Breaking Change)
### Removed
- Dropped OpenSearch support (Breaking Change)
- Dropped `HttpClientTransport` (Breaking Change)
- All depreacted code from 1.x, see [UPGRADE.md](UPGRADE.md) for the full list

@sfmok
Copy link
Contributor Author

sfmok commented Nov 20, 2024

Looks great thanks!

Can you add a note in the CHANGELOG?

A new "Fixed" section here please:

elastically/CHANGELOG.md

Lines 10 to 22 in 1bbd506

## 2.0.0
Massive release with added support for Elastica 8 and all deprecated code removed.
### Added
- Upgraded `ruflin/elastica` to v8 (Breaking Change)
### Removed
- Dropped OpenSearch support (Breaking Change)
- Dropped `HttpClientTransport` (Breaking Change)
- All depreacted code from 1.x, see [UPGRADE.md](UPGRADE.md) for the full list

Done

@damienalexandre damienalexandre merged commit c4bd889 into jolicode:master Nov 20, 2024
@sfmok sfmok deleted the bug/fix-denormalize-multiple-indexation branch November 20, 2024 23:53
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.

2 participants