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

feat: schema field level usage analytics #174

Merged
merged 111 commits into from
Oct 25, 2023
Merged

feat: schema field level usage analytics #174

merged 111 commits into from
Oct 25, 2023

Conversation

StarpTech
Copy link
Contributor

@StarpTech StarpTech commented Oct 9, 2023 •
edited
Loading

Motivation and Context

TODO

  • Store subgraph sourceIDs in clickhouse
  • Add service to Helm chart
  • Add documentation
  • Add materialized view
  • Set SourceID's correctly
  • Add service to docker-compose, demo
  • Integrate service with release pipeline

Motivation and Context

Linear

Fixes ENG-4370

@github-actions github-actions bot added the cli label Oct 24, 2023
@thisisnithin thisisnithin changed the title feat: implement GraphQL metrics collector feat: schema field level usage analytics Oct 25, 2023
thisisnithin and others added 19 commits October 25, 2023 07:40
@thisisnithin
fix ui
d14da40
@thisisnithin
fix ui layouts
2062ab3
@thisisnithin
loop command
fce09ca
@thisisnithin
fix empty operation name
1aaf9b5
@thisisnithin
query filter fix
e042105
@thisisnithin
fix import
e99a0d5
@thisisnithin
update date picker behavior
6a3b4d1
@thisisnithin
update datepicker ui
1e74d57
@thisisnithin
fix pagination
695dd45
@thisisnithin
unset sort on table relink
7b73d87
@thisisnithin
fix datepicker state
c99d3bb
@thisisnithin
fix metrics page range and operation trace link
15d8488
@thisisnithin
dry code
dc73528
@thisisnithin
fix empty results not fetching filters
3af832c
@StarpTech
Merge branch 'main' into dustin/eng-4276-implement-resolve-stats-coll…
a51633b 
…ector

# Conflicts:
#	router/core/context.go
#	router/core/graphql_handler.go
#	router/core/graphql_prehandler.go
#	router/core/operation_metrics.go
#	router/core/websocket.go
@StarpTech
chore: improve
57c9b83
@StarpTech
chore: improve
4013524
@StarpTech
Merge branch 'main' into dustin/eng-4276-implement-resolve-stats-coll…
a2f6f8c 
…ector
@thisisnithin
fix ui
079df17
fiam
fiam approved these changes Oct 25, 2023
View reviewed changes
Copy link
Contributor

@fiam fiam left a comment

Choose a reason for hiding this comment

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

LGTM

@StarpTech StarpTech dismissed jensneuse’s stale review October 25, 2023 17:45

I can't find the unresolved comments. All of them are resolved.

@StarpTech StarpTech merged commit 4f257a7 into main Oct 25, 2023
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fiam fiam fiam approved these changes

@devsergiy devsergiy Awaiting requested review from devsergiy

@jensneuse jensneuse Awaiting requested review from jensneuse

Assignees
No one assigned
Labels
cli connect controlplane monorepo protocol router studio
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@StarpTech @fiam @jensneuse @Aenimus @thisisnithin