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

[Backport 2.x] [BUG] Wrong breadcrumbs after browser reload on edit detectors page #395 #417

Merged
merged 1 commit into from
Feb 17, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 0aa6dd3 from #405

 (#405)

Signed-off-by: Jovan Cvetkovic <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit 0aa6dd3)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team February 17, 2023 22:17
@codecov-commenter
Copy link

Codecov Report

Merging #417 (8e0d3f9) into 2.x (b96450e) will increase coverage by 0.01%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##              2.x     #417      +/-   ##
==========================================
+ Coverage   15.40%   15.42%   +0.01%     
==========================================
  Files         112      112              
  Lines        3044     3060      +16     
  Branches      503      473      -30     
==========================================
+ Hits          469      472       +3     
- Misses       2536     2549      +13     
  Partials       39       39              
Impacted Files Coverage Δ
...nts/ConfigureAlerts/containers/ConfigureAlerts.tsx 6.66% <0.00%> (-1.03%) ⬇️
...ts/UpdateAlertConditions/UpdateAlertConditions.tsx 10.29% <0.00%> (+3.39%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@amsiglan amsiglan merged commit 2180730 into 2.x Feb 17, 2023
AWSHurneyt pushed a commit to AWSHurneyt/security-analytics-dashboards-plugin that referenced this pull request Feb 22, 2023
…pensearch-project#395 (opensearch-project#405) (opensearch-project#417)

Signed-off-by: Jovan Cvetkovic <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit 0aa6dd3)

Co-authored-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
AWSHurneyt pushed a commit to AWSHurneyt/security-analytics-dashboards-plugin that referenced this pull request Oct 12, 2023
…pensearch-project#395 (opensearch-project#405) (opensearch-project#417)

Signed-off-by: Jovan Cvetkovic <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit 0aa6dd3)

Co-authored-by: Jovan Cvetkovic <[email protected]>
Signed-off-by: AWSHurneyt <[email protected]>
amsiglan pushed a commit that referenced this pull request Mar 16, 2024
 (#405) (#417)

Signed-off-by: Jovan Cvetkovic <[email protected]>
Co-authored-by: Amardeepsingh Siglani <[email protected]>
(cherry picked from commit 0aa6dd3)

Co-authored-by: Jovan Cvetkovic <[email protected]>
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.

3 participants