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.

Sign up for GitHub

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

Jump to bottom

[OSCI] Remove ftr test from PR template #5215

Closed
wants to merge 6 commits into from
Closed

[OSCI] Remove ftr test from PR template #5215

wants to merge 6 commits into from

Conversation

Leo7Deng
Copy link
Contributor

@Leo7Deng Leo7Deng commented Oct 4, 2023 •
edited
Loading

Description

Removed ftr test step from PR template

Issues Resolved

fixes #5210

Screenshot

Screen Shot 2023-10-04 at 1 41 52 PM

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
    • yarn test:ftr
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

BSFishy and others added 6 commits October 4, 2023 15:52
@BSFishy @Leo7Deng
Reoder dev guide for yarn install to work (opensearch-project#5138)
6080149 
Signed-off-by: Matt Provost <[email protected]>
Signed-off-by: Leo Deng <[email protected]>
@AMoo-Miki @joshuarrrr @Leo7Deng
Bump agentkeepalive to v4.5.0 to solve a problem preventing the use…
d73a959 
… `https://ip` in `opensearch.hosts` (opensearch-project#4949)

The `elasticsearch` client uses a flaws version that incorrectly sets `servername` on a `tls` connection using IPs.

Signed-off-by: Miki <[email protected]>
Signed-off-by: Josh Romero <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Signed-off-by: Leo Deng <[email protected]>
@AMoo-Miki @Leo7Deng
[Console] Enhance support for JSON with long numerals (opensearch-pro…
ee7730b 
…ject#5130)

* Improve testing of clients with long numerals support
* Work around serializer inheritance in the JS client

Signed-off-by: Miki <[email protected]>
Signed-off-by: Leo Deng <[email protected]>
@ananzh @AMoo-Miki
@ashwin-pc @Leo7Deng
[BUG][Discover] Add onQuerySubmit to top nav and allow force update t…
1e93ed4 
…o embeddable (opensearch-project#5160)

* all reload to force update embeddable
* add onQuerySubmit to top nav

Issue Resolve
opensearch-project#5116
opensearch-project#5159

Signed-off-by: ananzh <[email protected]>
Co-authored-by: Miki <[email protected]>
Co-authored-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Leo Deng <[email protected]>
@abbyhu2000 @AMoo-Miki
@joshuarrrr @Leo7Deng
fix removing pinned global filters (opensearch-project#5143)
2c1f71d 
Signed-off-by: abbyhu2000 <[email protected]>
Co-authored-by: Miki <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Signed-off-by: Leo Deng <[email protected]>
@ashwin-pc @Leo7Deng
[Discover] A bunch of navigation fixes (opensearch-project#5168)
7d84753 
* Discover: Fixes state persistence after nav
* Fixed breadcrumbs and navigation
* fixes mobile view

---------

Signed-off-by: Ashwin P Chandran <[email protected]>
Signed-off-by: Leo Deng <[email protected]>
@Leo7Deng Leo7Deng closed this Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ananzh ananzh Awaiting requested review from ananzh ananzh is a code owner

@kavilla kavilla Awaiting requested review from kavilla kavilla is a code owner

@seanneumann seanneumann Awaiting requested review from seanneumann

@AMoo-Miki AMoo-Miki Awaiting requested review from AMoo-Miki AMoo-Miki is a code owner

@ashwin-pc ashwin-pc Awaiting requested review from ashwin-pc ashwin-pc is a code owner

@joshuarrrr joshuarrrr Awaiting requested review from joshuarrrr joshuarrrr is a code owner

@abbyhu2000 abbyhu2000 Awaiting requested review from abbyhu2000 abbyhu2000 is a code owner

@zengyan-amazon zengyan-amazon Awaiting requested review from zengyan-amazon zengyan-amazon is a code owner

@kristenTian kristenTian Awaiting requested review from kristenTian

@zhongnansu zhongnansu Awaiting requested review from zhongnansu zhongnansu is a code owner

@manasvinibs manasvinibs Awaiting requested review from manasvinibs manasvinibs is a code owner

@ZilongX ZilongX Awaiting requested review from ZilongX ZilongX is a code owner

@Flyingliuhub Flyingliuhub Awaiting requested review from Flyingliuhub Flyingliuhub is a code owner

@BSFishy BSFishy Awaiting requested review from BSFishy

@curq curq Awaiting requested review from curq curq is a code owner

Assignees
No one assigned
Labels
first-time-contributor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Remove ftr test step from PR template
6 participants
@Leo7Deng @BSFishy @AMoo-Miki @ananzh @abbyhu2000 @ashwin-pc