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

feat(elasticsearch-plugin): provide the ctx for custom mappings #2547

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

mcfedr
Copy link
Contributor

@mcfedr mcfedr commented Nov 24, 2023

Description

Given custom mapping access to the ctx, so they can add channel specific data

Breaking changes

No

Screenshots

N/A

Checklist

📌 Always:

  • I have set a clear title
  • My PR is small and contains a single feature
  • I have checked my own PR

⚡ Most of the time:

  • I have added or updated test cases
  • I have updated the README if needed

Copy link

netlify bot commented Nov 24, 2023

Deploy Preview for effervescent-donut-4977b2 canceled.

Name Link
🔨 Latest commit 8d2ffc6
🔍 Latest deploy log https://app.netlify.com/sites/effervescent-donut-4977b2/deploys/6560a2801089790008a30f5f

@michaelbromley
Copy link
Member

Hi @mcfedr - nice to hear from you, it has been a while!

This PR is a good additional - please just make it against the minor branch as it is a new feature 👍

@mcfedr mcfedr changed the base branch from master to minor November 27, 2023 09:24
@mcfedr
Copy link
Contributor Author

mcfedr commented Nov 27, 2023

@michaelbromley updated the target branch

@michaelbromley michaelbromley merged commit c5d0ea2 into vendure-ecommerce:minor Nov 27, 2023
16 of 18 checks passed
@michaelbromley
Copy link
Member

Thank you!

@mcfedr
Copy link
Contributor Author

mcfedr commented Dec 14, 2023

@michaelbromley any chance we could get a release with this?

@michaelbromley
Copy link
Member

@mcfedr I just published v2.2.0-next.0 👍

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