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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

views: Improve API ergonomy #148

Merged
merged 6 commits into from
Jul 6, 2023
Merged

views: Improve API ergonomy #148

merged 6 commits into from
Jul 6, 2023

Conversation

victorges
Copy link
Member

@victorges victorges commented Jun 28, 2023

  • Separate null from not-asked fields
  • Fix "exists before starts" typo 馃き
  • Allow querying global account metrics
  • added viewerId and creatorId to breakdownBy fields

@victorges victorges requested a review from a team as a code owner June 28, 2023 17:56
@codecov
Copy link

codecov bot commented Jun 28, 2023

Codecov Report

Merging #148 (fad0ffd) into main (20c9d93) will increase coverage by 1.39929%.
The diff coverage is 10.63830%.

Impacted file tree graph

@@                 Coverage Diff                 @@
##                main        #148         +/-   ##
===================================================
+ Coverage   26.53722%   27.93651%   +1.39929%     
===================================================
  Files              4           4                 
  Lines            309         315          +6     
===================================================
+ Hits              82          88          +6     
- Misses           215         217          +2     
+ Partials          12          10          -2     
Impacted Files Coverage 螖
views/client.go 0.00000% <0.00000%> (酶)
views/bigquery.go 49.29577% <71.42857%> (+0.74505%) 猬嗭笍

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
螖 = absolute <relative> (impact), 酶 = not affected, ? = missing data
Powered by Codecov. Last update 20c9d93...fad0ffd. Read the comment docs.

Impacted Files Coverage 螖
views/client.go 0.00000% <0.00000%> (酶)
views/bigquery.go 49.29577% <71.42857%> (+0.74505%) 猬嗭笍

... and 1 file with indirect coverage changes

views/client.go Outdated Show resolved Hide resolved
views/client.go Outdated Show resolved Hide resolved
Copy link
Member

@ecmulli ecmulli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure to replace all ExistsBeforeStart with ExitBeforeStart

ecmulli and others added 2 commits July 5, 2023 16:11
* added viewerid and creatorid to the events query

* Update views/client.go

Co-authored-by: Victor Elias <victor@livepeer.org>

* updated to casing

---------

Co-authored-by: Victor Elias <victor@livepeer.org>
* added viewerid and creatorid to the events query

* Update views/client.go

Co-authored-by: Victor Elias <victor@livepeer.org>

* updated to casing

* added geohash

---------

Co-authored-by: Victor Elias <victor@livepeer.org>
Copy link
Member

@gioelecerati gioelecerati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@victorges victorges merged commit b98d351 into main Jul 6, 2023
13 checks passed
@victorges victorges deleted the vg/feat/cleaner-api branch July 6, 2023 14:37
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.

None yet

3 participants