-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Render the correct form when edit extraction rules #151304
Conversation
It doesn't fix the issue completely I believe. I managed to get 4 extraction rules that have same name and different field configurations this way. |
5841636
to
a21f8cb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me. One nitpick
...nents/search_index/crawler/crawler_domain_detail/extraction_rules/extraction_rules_logic.tsx
Outdated
Show resolved
Hide resolved
💚 Build Succeeded
Metrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
Adjust Kea's reducers to render correct HTML forms (cherry picked from commit 1bba627)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…151480) # Backport This will backport the following commits from `main` to `8.7`: - [Render the correct form when edit extraction rules (#151304)](#151304) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Dmitriy Burlutskiy","email":"dmitrii.burlutckii@elastic.co"},"sourceCommit":{"committedDate":"2023-02-16T16:11:01Z","message":"Render the correct form when edit extraction rules (#151304)\n\nAdjust Kea's reducers to render correct HTML forms","sha":"1bba627926d944456da2a5189242162eb2a7ec59","branchLabelMapping":{"^v8.8.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v8.7.0","v8.8.0"],"number":151304,"url":"#151304 the correct form when edit extraction rules (#151304)\n\nAdjust Kea's reducers to render correct HTML forms","sha":"1bba627926d944456da2a5189242162eb2a7ec59"}},"sourceBranch":"main","suggestedTargetBranches":["8.7"],"targetPullRequestStates":[{"branch":"8.7","label":"v8.7.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.8.0","labelRegex":"^v8.8.0$","isSourceBranch":true,"state":"MERGED","url":"#151304 the correct form when edit extraction rules (#151304)\n\nAdjust Kea's reducers to render correct HTML forms","sha":"1bba627926d944456da2a5189242162eb2a7ec59"}}]}] BACKPORT--> Co-authored-by: Dmitriy Burlutskiy <dmitrii.burlutckii@elastic.co>
Closes https://github.com/elastic/enterprise-search-team/issues/3934
Summary
The wrong form (
create
instead ofedit
) is rendered when I try to modify an extraction rule.