Skip to content

feat: add contains ignore case in go to row#1291

Merged
ethanalvizo merged 4 commits intodeephaven:mainfrom
ethanalvizo:1274-new-seek-row-option
May 17, 2023
Merged

feat: add contains ignore case in go to row#1291
ethanalvizo merged 4 commits intodeephaven:mainfrom
ethanalvizo:1274-new-seek-row-option

Conversation

@ethanalvizo
Copy link
Copy Markdown
Contributor

Closes #1274

@ethanalvizo ethanalvizo self-assigned this May 11, 2023
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 11, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@ethanalvizo ethanalvizo requested a review from mofojed May 11, 2023 15:46
Comment thread packages/iris-grid/src/GotoRow.tsx
Comment thread packages/iris-grid/src/IrisGrid.tsx
Comment thread packages/iris-grid/src/GotoRow.test.tsx
@ethanalvizo ethanalvizo force-pushed the 1274-new-seek-row-option branch from 33af2df to 2ba0f6c Compare May 11, 2023 19:47
@ethanalvizo
Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@ethanalvizo
Copy link
Copy Markdown
Contributor Author

recheck

@ethanalvizo ethanalvizo force-pushed the 1274-new-seek-row-option branch from 2ba0f6c to 4baa67e Compare May 11, 2023 21:05
@codecov
Copy link
Copy Markdown

codecov Bot commented May 11, 2023

Codecov Report

Merging #1291 (c3ad3f1) into main (d304300) will decrease coverage by 0.35%.
The diff coverage is 20.00%.

❗ Current head c3ad3f1 differs from pull request most recent head a2273e0. Consider uploading reports for the commit a2273e0 to get more accurate results

@@            Coverage Diff             @@
##             main    #1291      +/-   ##
==========================================
- Coverage   45.89%   45.54%   -0.35%     
==========================================
  Files         492      504      +12     
  Lines       34371    34827     +456     
  Branches     8566     8690     +124     
==========================================
+ Hits        15775    15863      +88     
- Misses      18545    18913     +368     
  Partials       51       51              
Flag Coverage Δ
unit 45.54% <20.00%> (-0.35%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
packages/iris-grid/src/IrisGrid.tsx 27.10% <0.00%> (+0.01%) ⬆️
packages/iris-grid/src/GotoRow.tsx 59.37% <100.00%> (+19.79%) ⬆️

... and 51 files with indirect coverage changes

mofojed
mofojed previously approved these changes May 12, 2023
Comment thread packages/iris-grid/src/GotoRow.tsx Outdated
Comment thread packages/iris-grid/src/GotoRow.tsx Outdated
@ethanalvizo ethanalvizo force-pushed the 1274-new-seek-row-option branch from 6635d6e to 90227c1 Compare May 12, 2023 19:19
Comment thread packages/iris-grid/src/IrisGrid.tsx
@ethanalvizo ethanalvizo force-pushed the 1274-new-seek-row-option branch from edd980e to c3ad3f1 Compare May 12, 2023 20:42
@ethanalvizo ethanalvizo force-pushed the 1274-new-seek-row-option branch from c3ad3f1 to d4c2ad9 Compare May 15, 2023 14:28
@ethanalvizo ethanalvizo requested a review from mofojed May 15, 2023 15:01
Comment thread packages/iris-grid/src/GotoRow.tsx Outdated
@ethanalvizo ethanalvizo requested a review from mofojed May 16, 2023 20:36
@ethanalvizo ethanalvizo merged commit d67712e into deephaven:main May 17, 2023
@github-actions github-actions Bot locked and limited conversation to collaborators May 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Contains Ignore Case option to Seek Row functionality

3 participants