-
Notifications
You must be signed in to change notification settings - Fork 98
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
Use mozfun.norm.fenix_app_info for nondesktop_clients_last_seen #1245
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jklukas
added a commit
to mozilla/gud
that referenced
this pull request
Aug 20, 2020
Should wait until mozilla/bigquery-etl#1245 is merged. The mapping from Fenix channels to appIds is no longer straight-forward, so we remove those shortDescription values.
fbertsch
approved these changes
Aug 20, 2020
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but I have a small reservation about the CTE naming scheme going on here :)
After this is merged, we can backfill of smoot tables to have GUD reflect the current and historical app naming schemes.
jklukas
force-pushed
the
gud-fenix-refactor
branch
from
August 21, 2020 13:20
e65cbfc
to
d6e599b
Compare
jklukas
added a commit
to mozilla/gud
that referenced
this pull request
Sep 9, 2020
Should wait until mozilla/bigquery-etl#1245 is merged. The mapping from Fenix channels to appIds is no longer straight-forward, so we remove those shortDescription values.
jklukas
added a commit
to mozilla/gud
that referenced
this pull request
Sep 9, 2020
Should wait until mozilla/bigquery-etl#1245 is merged. The mapping from Fenix channels to appIds is no longer straight-forward, so we remove those shortDescription values.
jklukas
added a commit
to mozilla/gud
that referenced
this pull request
Sep 9, 2020
Should wait until mozilla/bigquery-etl#1245 is merged. The mapping from Fenix channels to appIds is no longer straight-forward, so we remove those shortDescription values.
jklukas
added a commit
to mozilla/gud
that referenced
this pull request
Sep 9, 2020
Should wait until mozilla/bigquery-etl#1245 is merged. The mapping from Fenix channels to appIds is no longer straight-forward, so we remove those shortDescription values.
jklukas
added a commit
to mozilla/gud
that referenced
this pull request
Sep 9, 2020
Should wait until mozilla/bigquery-etl#1245 is merged. The mapping from Fenix channels to appIds is no longer straight-forward, so we remove those shortDescription values.
jklukas
added a commit
to mozilla/gud
that referenced
this pull request
Jan 6, 2021
* GUD v2 (graph-paper revamp) (#85) * adds barebones from graph-paper story * integrates config file * adds back Logo; further tweaks to menu items; implements cache + querying * adds Loading Spinner * finesses error / loading states * adds calls to action (feedback, shortcuts, docs, file an issue) in Sidebar * adds esc * further tweaks to Shortcuts menu * styling tweaks * adds back querystring state loading * updates store to have a no-local querying approach to support transitions within Explore.svelte * adds back in TimeAxis.svelte * updates and tweaks some layout pieces * updates the mouseovers to account for bug fixes (contingent on a new graph-paper PR) * adds back in caveat system * adds smoothing option * adds hash routing for future improvements * fixes the hashbang routing with query parameteres * wires up explore body menus to read from querystring * metric selector; moves some compoinents to src/components/controls * compartmentalizes the mouse interactions * adds back in markers * cleans up modal implementatino * sidebar tweaks * adds a few more affordances around YoY + mouseovers; cleans up all menus * feature-complete * adds disabledMetrics and disabledDimensions * makes lists keyboard-accessible * restyles sidebar in anticipation of multiple views * migrates to graph-paper 0.0.0-alpha.14 * improves mouseover performance * further design tweaks * adds in changes to usage criterion from prototype branch * adds newer clickOutside action * reset start and end dates on usage criterion change * removes extraneous comments * updates a handful of dependencies * adds back tooltips into graphs * adds TitleMenu.svelte component * Consolidates routing / config information * adds disabled functionality * adds rudimeentary scrollIntoView support * adds CHANGELOG.md * date picker closes after apply is clicked (#97) * moves around a few components; fixes DatePicker animation issue (#102) * moves around a few components; fixes DatePicker animation issue * collapses DatePicker on reset * removes buttons in explore when focused on a single metric * Add basic front-end CI tests (#103) * ESLint improvements (#105) * Switch to a Node-based eslint-health script * Run ESLint against Svelte files * Update eslint-health.js with small docs changes * Remove appId labels from Fenix channels (#107) Should wait until mozilla/bigquery-etl#1245 is merged. The mapping from Fenix channels to appIds is no longer straight-forward, so we remove those shortDescription values. * bugfixes before fbo tomorrow (#111) * Add Japan to the country list (#114) Resolves #112 Relies on mozilla/bigquery-etl#1352 to make JP available in the underlying tables. * Never use "Fennec" for "Firefox for iOS" (#120) * Remove references to folks no longer involved (#125) * redirects / to /explore, which fixes future instantiation issues because the route is not codified in the hash. (#124) * uses labels instead of keys for sidebar and body title (#123) * Remove usage criteria docs, which are out of date (#126) This same documentation exists in the UI, so I think it's best to remove it here. * Adjust to modified product names (#128) See mozilla/bigquery-etl#1380 Co-authored-by: Hamilton Ulmer <hamilton.ulmer@gmail.com> Co-authored-by: John Karahalis <john.karahalis@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
After this is merged, we can backfill of smoot tables to have GUD reflect the current and historical app naming schemes.
Note that we are not updating the incline executive dashboard query to match this. In consulation with @magorlick, we decided that changing behavior of the incline dash at this point would be more disruptive than helpful. The dashboard is mostly all focused on release at this point, anyhow, which is not affected by this change in channel definitions.