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

Better inline filter #596

Merged

Conversation

edlouth
Copy link
Contributor

@edlouth edlouth commented Aug 21, 2023

No description provided.

@edlouth edlouth changed the base branch from master to feature/gra-682-graphfilter-live-create-and-edit August 21, 2023 13:01
@github-actions
Copy link
Contributor

github-actions bot commented Aug 21, 2023

Uffizzi Preview deployment-33955 was deleted.

@codecov
Copy link

codecov bot commented Aug 21, 2023

Codecov Report

Patch coverage: 51.86% and project coverage change: -2.13% ⚠️

Comparison is base (c184777) 97.18% compared to head (787e8e1) 95.06%.
Report is 1 commits behind head on feature/gra-682-graphfilter-live-create-and-edit.

Additional details and impacted files
@@                                 Coverage Diff                                  @@
##           feature/gra-682-graphfilter-live-create-and-edit     #596      +/-   ##
====================================================================================
- Coverage                                             97.18%   95.06%   -2.13%     
====================================================================================
  Files                                                   385      394       +9     
  Lines                                                  7072     7330     +258     
  Branches                                               1043     1163     +120     
====================================================================================
+ Hits                                                   6873     6968      +95     
- Misses                                                  196      359     +163     
  Partials                                                  3        3              
Flag Coverage Δ
grai-frontend 92.75% <39.35%> (-4.74%) ⬇️
grai-server 97.08% <100.00%> (+0.14%) ⬆️

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

Files Changed Coverage Δ
...tend/src/components/auth/register/RegisterCard.tsx 100.00% <ø> (ø)
...i-frontend/src/components/filters/CreateFilter.tsx 100.00% <ø> (ø)
...rai-frontend/src/components/filters/FilterForm.tsx 100.00% <ø> (ø)
...rai-frontend/src/components/filters/FilterRows.tsx 100.00% <ø> (ø)
...i-frontend/src/components/filters/UpdateFilter.tsx 100.00% <ø> (ø)
grai-frontend/src/components/graph/BaseGraph.tsx 45.71% <ø> (ø)
...i-frontend/src/components/graph/DataSourceIcon.tsx 100.00% <ø> (ø)
...i-frontend/src/components/graph/GraphComponent.tsx 67.41% <ø> (ø)
...ponents/graph/drawer/filters-inline/FilterItem.tsx 3.22% <3.22%> (ø)
...nents/graph/drawer/filters-inline/FilterPopper.tsx 4.39% <4.39%> (ø)
... and 20 more

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@edlouth edlouth merged commit e7050c9 into feature/gra-682-graphfilter-live-create-and-edit Aug 21, 2023
38 of 41 checks passed
@edlouth edlouth deleted the better-inline-filter branch August 21, 2023 14:20
edlouth added a commit that referenced this pull request Aug 23, 2023
* Start on live edit

* Better inline filter (#596)

* Looker integration (#537)

* Looker alpha release

---------

Co-authored-by: Ian Eaves <ian.k.eaves@gmail.com>

* Better inline filter

* Partially working inline filters

* Pretty much working frontend

* Fix links (#593)

* Fix cron hours (#594)

* Fix looker icon (#595)

* Add inline filters to graph type

* Inline filters

* Lint fixes

* Backend tests

---------

Co-authored-by: Ian Eaves <ian.k.eaves@gmail.com>

* Frontend refactor

* More tests

* Fix code style issues with ESLint

* Some more bits and pieces

* Lint fixes

* Remove acts

* More tests

* More frontend tests

* Fix code style issues with ESLint

* Refactor and tests

* Fix for tags

* Additional test

* Test fixes

* Add test

---------

Co-authored-by: Ian Eaves <ian.k.eaves@gmail.com>
Co-authored-by: Lint Action <github-actions[bot]@users.noreply.github.com>
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.

1 participant