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

fix: update to latest giraffe to fix font issues #5843

Merged
merged 1 commit into from
Sep 21, 2022

Conversation

hoorayimhelping
Copy link
Contributor

Closes #1673
Closes #2629

Closes influxdata/giraffe#811

Pulls in the latest giraffe which adds the following fixes:

  • fixes font sanitizer console warnings
  • Gauge and SingleStat/SingleStat with Line graphs now use Proxima Nova instead of the default system serif font

Checklist

Authors and Reviewer(s), please verify the following:

  • A PR description, regardless of the triviality of this change, that communicates the value of this PR
  • Well-formatted conventional commit messages that provide context into the change
  • Documentation updated or issue created (provide link to issue/PR)
  • [ ] Signed CLA (if not already signed)
  • [ ] Feature flagged, if applicable

@hoorayimhelping hoorayimhelping requested a review from a team as a code owner September 21, 2022 15:19
@hoorayimhelping hoorayimhelping requested a review from a team September 21, 2022 15:19
@hoorayimhelping hoorayimhelping added this pull request to the merge queue Sep 21, 2022
Merged via the queue into master with commit f7bda80 Sep 21, 2022
@hoorayimhelping hoorayimhelping deleted the bucky_fix_giraffe_fonts branch September 21, 2022 16:00
subirjolly pushed a commit that referenced this pull request Sep 23, 2022
chore: rename function

chore: remove unused import

chore: update per review

chore: refactor to remove index out of bounds issue

chore: direct state change without mutator per review

chore: minor changes and add more status code checks to pactch account response

chore: update userAccounts with updated reference

chore: remove not required code

chore: remove response status case which is highly unlikely

chore: remove unnecessary activeAccount check

chore: remove unused import

feat: use quartz, not idpe, for user sessions (#5769)

* fix: use quartz for sessions behind feat flag

* chore: fix relative import of feature flag

* chore: remove unnecessary comment

feat(5340): use default editorText in script editor (#5741)

* feat(5340): use default editorText in script editor; updates to session, new script resource, and clear() methods

* feat(5340): update tests for new default text, and caught edge case for certain flag+action combination

* test: demonstrate that we need the default query set on both the session, and in the script resource. In order to have on-load provide the default message.

* fix: add back the default message, to the default session store

fix(schemaComposition): make highlight to scroll with editor (#5719)

* style: use native monaco editor api for customized style for sync highlight

* chore: run prettier

* chore: add sync off style

* chore: add clickable icon to editor

* chore: update comment

* chore: make sync icon to be a background image and un-clickable

* chore: run prettier

* chore: add comment

* test: update classNames in test

chore: remove avatarWidgetMultiAccountInfo feature flag code (#5803)

* chore: remove avatarWidgetMultiAccountInfo feature flag code

* chore: remove unused imports

Co-authored-by: Subir Jolly <[email protected]>

feat: sync updated account name across header and profile page.

chore: remove unused imports

chore: remove globalheadercontainer

chore: remove not used code

chore: add cloud check fir using useraccountprovider

chore: update to have cloud check for user account provider

chore: add guards to pushing into selected variable array, remove extra reporting (#5791)

* chore: replace not-extensible honeybadger reporting with a console.error to reduce network calls

* chore: remove unused import

* fix: add guard to not push to vari.selected if not extensible

chore(CsvMethod): move context logic into CsvMethod (#5751)

* chore: move context logic into CsvMethod

* fix: prettier

* fix: prettier

* chore: remove csvUploader file

* chore: clean up

chore(5780): track usage of fallback worker (#5782)

feat(idpe-15219): remove pinned items from dashboards (#5789)

feat(idpe-15219): remove pinned items from cypress tests (#5785)

chore: upgrade normalizr to latest (#5792)

chore: remove http-proxy-middleware (#5749)

* chore: upgrade http-proxy-middleware to the latest

* chore: remove http-proxy-middleware

feat(idpe-15219): remove pinnedItems components, no longer used on homepage (#5786)

style: remove the corner radius of the Monaco editor and Results panel (#5772)

* style: remove the border radius for Monaco editor and Results pane

* style: remove margin bottom and update border size for the heading

* style: add top padding for two side bars on the left and right

* style: increase padding to the heading menu

chore: Upgrade chroma-js and it's types to latest (#5798)

* chore: upgrade @types/chroma-js to latest

* chore: upgrade chroma-js to latest

chore: upgrade monaco to latest (#5793)

* chore: upgrade monaco to latest

* chore: change monaco language client to 2.1.0

* chore: change monaco language client to 0.18.1

* chore: tilda for react monaco editor

* chore: yarn lock

chore: upgrade @types/chai to latest (#5797)

fix: page content now visible on mobile devices (#5651)

* fix: page content now visible on mobile devices

* chore: linter

* fix: global header popover

* fix: remove custom style adjustments for dapperscrollbar, dataexplorer, tasks

* fix: remove custom style adjustments for dapperscrollbar, dataexplorer, tasks

Co-authored-by: wdoconnell <[email protected]>

feat(idpe-15219): remove pinned items from Tasks (#5784)

chore: upgrade testing-library to latest (#5795)

* chore: upgrade testing-library to latest

* chore: downgrade testing-library/react to 12
.1.5

fix(sub): password field optional during update (#5812)

feat(idpe-15219): remove pinned items from flows (#5783)

fix(firstMile/arduino): font size for "Note" (#5823)

feat: add multiselect for measurements (#5673)

* feat: add multiselect for measurements

* fix: missed some edgecase behaviors

* fix: updating styling and text per design review

Co-authored-by: drdelambre <drdelambre>

feat(idpe-15219): remove pinnedItems context and notification methods (#5822)

chore: refactor useraccounts to use api layer (#5773)

* chore: refactor useraccounts to use api layer

* chore: rename function

* chore: remove unused import

* chore: update per review

* chore: refactor to remove index out of bounds issue

* chore: direct state change without mutator per review

* chore: minor changes and add more status code checks to pactch account response

* chore: update userAccounts with updated reference

* chore: remove not required code

* chore: remove response status case which is highly unlikely

* chore: remove unnecessary activeAccount check

* chore: remove unused import

* chore: remove unused error notification

* chore: reporting account errors to honebadger

* chore: update payload info for reportErrorThroughHoneyBadger

* chore: update per review

Co-authored-by: Subir Jolly <[email protected]>

chore: upgrade auth0-js to latest (#5796)

chore: fix array length check

chore: use optional chaining for getting length

chore: update name of action function

chore: fix imports path

chore: add account settings page behind cloud

feat: replace isRegionBeta with isUpgradeable from identity (#5808)

chore: update jest-junit (#5827)

fix(firstMile/arduino): scroll to selected bucket (#5825)

chore: upgrade css-loader to v5.2.7 (#5747)

* chore: upgrade css-loader to the latest

* chore: upgrade css-loader to the v5.2.7

* chore: yarn lock

fix: change expected acct endpoint response to single object (#5838)

fix(firstMile): fixing listen data component to listen on component mount (#5830)

* fix(firstMile): fixing listen data component to listen on componentDidMount as well

* fix(firstMile): make a function for reinitializing listening

* fix(firstMile): import

chore: update Giraffe (#5839)

fix(rudderstack): update version to remove app flicker (#5841)

feat: add new Telegraf plugins and update existing plugins (#5842)

feat: add user feedback button (#5831)

* feat: add userpilot feedback button in new script editor

* chore: add feature flag around the feedback button

fix: update to latest giraffe to fix font issues (#5843)

feat: add new time range component for the UI for the new data explorer (#5817)

* wip

* feat: added validation, need to clean up design when they are ready and clean up the component logic

* feat: connect all the things

fix: use react router links in lieu of redirect for gh main menu subelements (#5828)

fix(reporting): guards to prevent accessing null values from useSelector (#5848)

chore: upgrade html2canvas (#5850)

chore: upgrade intersection-observer polyfill (#5853)

fix: if no new cert is provided a user should be able to submit (#5852)

* fix: if no new cert is provided a user should be able to update their sub

* fix: lint

* fix: prettier

chore: upgrade clockface to 6.3.5 (#5854)

chore: remove ability to use quartzMe (#5847)

chore: fix component name

fix: update save script text (#5857)

Co-authored-by: drdelambre <drdelambre>

fix: kick out users if new DE flag is off (#5858)

* fix: let users leave the experiment

* fix: kick them out even more

* fix: force flag to be set on every test

Co-authored-by: drdelambre <drdelambre>

chore(dependencies): update to clockface 6.3.6 to attempt fixing corrupt icon fonts (#5863)

chore: convert string id to number

chore: change currentaccount to account type

chore: update icon for Open and Save (#5860)

fix: add caching to color visualization calculation (#5851)

chore: update to use Number function and use relative import

chore: remove original injection commands (#5840)

* chore: remove injectViaLsp() for measurement, field, and tag values

* chore: remove ExecuteCommandInjectField

* chore: remove ExecuteCommand.InjectField

* chore: remove ExecuteCommandInjectTag and ExecuteCommandInjectTagValue

* chore: remove ExecuteCommandInjectMeasurement

* chore: remove injectViaLsp() implementation

fix: resolve flakiness by filling gaps in call to makeQuartzUseIDPEOrgID() (#5871)
subirjolly pushed a commit that referenced this pull request Sep 23, 2022
chore: rename function

chore: remove unused import

chore: update per review

chore: refactor to remove index out of bounds issue

chore: direct state change without mutator per review

chore: minor changes and add more status code checks to pactch account response

chore: update userAccounts with updated reference

chore: remove not required code

chore: remove response status case which is highly unlikely

chore: remove unnecessary activeAccount check

chore: remove unused import

feat: use quartz, not idpe, for user sessions (#5769)

* fix: use quartz for sessions behind feat flag

* chore: fix relative import of feature flag

* chore: remove unnecessary comment

feat(5340): use default editorText in script editor (#5741)

* feat(5340): use default editorText in script editor; updates to session, new script resource, and clear() methods

* feat(5340): update tests for new default text, and caught edge case for certain flag+action combination

* test: demonstrate that we need the default query set on both the session, and in the script resource. In order to have on-load provide the default message.

* fix: add back the default message, to the default session store

fix(schemaComposition): make highlight to scroll with editor (#5719)

* style: use native monaco editor api for customized style for sync highlight

* chore: run prettier

* chore: add sync off style

* chore: add clickable icon to editor

* chore: update comment

* chore: make sync icon to be a background image and un-clickable

* chore: run prettier

* chore: add comment

* test: update classNames in test

chore: remove avatarWidgetMultiAccountInfo feature flag code (#5803)

* chore: remove avatarWidgetMultiAccountInfo feature flag code

* chore: remove unused imports

Co-authored-by: Subir Jolly <[email protected]>

feat: sync updated account name across header and profile page.

chore: remove unused imports

chore: remove globalheadercontainer

chore: remove not used code

chore: add cloud check fir using useraccountprovider

chore: update to have cloud check for user account provider

chore: add guards to pushing into selected variable array, remove extra reporting (#5791)

* chore: replace not-extensible honeybadger reporting with a console.error to reduce network calls

* chore: remove unused import

* fix: add guard to not push to vari.selected if not extensible

chore(CsvMethod): move context logic into CsvMethod (#5751)

* chore: move context logic into CsvMethod

* fix: prettier

* fix: prettier

* chore: remove csvUploader file

* chore: clean up

chore(5780): track usage of fallback worker (#5782)

feat(idpe-15219): remove pinned items from dashboards (#5789)

feat(idpe-15219): remove pinned items from cypress tests (#5785)

chore: upgrade normalizr to latest (#5792)

chore: remove http-proxy-middleware (#5749)

* chore: upgrade http-proxy-middleware to the latest

* chore: remove http-proxy-middleware

feat(idpe-15219): remove pinnedItems components, no longer used on homepage (#5786)

style: remove the corner radius of the Monaco editor and Results panel (#5772)

* style: remove the border radius for Monaco editor and Results pane

* style: remove margin bottom and update border size for the heading

* style: add top padding for two side bars on the left and right

* style: increase padding to the heading menu

chore: Upgrade chroma-js and it's types to latest (#5798)

* chore: upgrade @types/chroma-js to latest

* chore: upgrade chroma-js to latest

chore: upgrade monaco to latest (#5793)

* chore: upgrade monaco to latest

* chore: change monaco language client to 2.1.0

* chore: change monaco language client to 0.18.1

* chore: tilda for react monaco editor

* chore: yarn lock

chore: upgrade @types/chai to latest (#5797)

fix: page content now visible on mobile devices (#5651)

* fix: page content now visible on mobile devices

* chore: linter

* fix: global header popover

* fix: remove custom style adjustments for dapperscrollbar, dataexplorer, tasks

* fix: remove custom style adjustments for dapperscrollbar, dataexplorer, tasks

Co-authored-by: wdoconnell <[email protected]>

feat(idpe-15219): remove pinned items from Tasks (#5784)

chore: upgrade testing-library to latest (#5795)

* chore: upgrade testing-library to latest

* chore: downgrade testing-library/react to 12
.1.5

fix(sub): password field optional during update (#5812)

feat(idpe-15219): remove pinned items from flows (#5783)

fix(firstMile/arduino): font size for "Note" (#5823)

feat: add multiselect for measurements (#5673)

* feat: add multiselect for measurements

* fix: missed some edgecase behaviors

* fix: updating styling and text per design review

Co-authored-by: drdelambre <drdelambre>

feat(idpe-15219): remove pinnedItems context and notification methods (#5822)

chore: refactor useraccounts to use api layer (#5773)

* chore: refactor useraccounts to use api layer

* chore: rename function

* chore: remove unused import

* chore: update per review

* chore: refactor to remove index out of bounds issue

* chore: direct state change without mutator per review

* chore: minor changes and add more status code checks to pactch account response

* chore: update userAccounts with updated reference

* chore: remove not required code

* chore: remove response status case which is highly unlikely

* chore: remove unnecessary activeAccount check

* chore: remove unused import

* chore: remove unused error notification

* chore: reporting account errors to honebadger

* chore: update payload info for reportErrorThroughHoneyBadger

* chore: update per review

Co-authored-by: Subir Jolly <[email protected]>

chore: upgrade auth0-js to latest (#5796)

chore: fix array length check

chore: use optional chaining for getting length

chore: update name of action function

chore: fix imports path

chore: add account settings page behind cloud

feat: replace isRegionBeta with isUpgradeable from identity (#5808)

chore: update jest-junit (#5827)

fix(firstMile/arduino): scroll to selected bucket (#5825)

chore: upgrade css-loader to v5.2.7 (#5747)

* chore: upgrade css-loader to the latest

* chore: upgrade css-loader to the v5.2.7

* chore: yarn lock

fix: change expected acct endpoint response to single object (#5838)

fix(firstMile): fixing listen data component to listen on component mount (#5830)

* fix(firstMile): fixing listen data component to listen on componentDidMount as well

* fix(firstMile): make a function for reinitializing listening

* fix(firstMile): import

chore: update Giraffe (#5839)

fix(rudderstack): update version to remove app flicker (#5841)

feat: add new Telegraf plugins and update existing plugins (#5842)

feat: add user feedback button (#5831)

* feat: add userpilot feedback button in new script editor

* chore: add feature flag around the feedback button

fix: update to latest giraffe to fix font issues (#5843)

feat: add new time range component for the UI for the new data explorer (#5817)

* wip

* feat: added validation, need to clean up design when they are ready and clean up the component logic

* feat: connect all the things

fix: use react router links in lieu of redirect for gh main menu subelements (#5828)

fix(reporting): guards to prevent accessing null values from useSelector (#5848)

chore: upgrade html2canvas (#5850)

chore: upgrade intersection-observer polyfill (#5853)

fix: if no new cert is provided a user should be able to submit (#5852)

* fix: if no new cert is provided a user should be able to update their sub

* fix: lint

* fix: prettier

chore: upgrade clockface to 6.3.5 (#5854)

chore: remove ability to use quartzMe (#5847)

chore: fix component name

fix: update save script text (#5857)

Co-authored-by: drdelambre <drdelambre>

fix: kick out users if new DE flag is off (#5858)

* fix: let users leave the experiment

* fix: kick them out even more

* fix: force flag to be set on every test

Co-authored-by: drdelambre <drdelambre>

chore(dependencies): update to clockface 6.3.6 to attempt fixing corrupt icon fonts (#5863)

chore: convert string id to number

chore: change currentaccount to account type

chore: update icon for Open and Save (#5860)

fix: add caching to color visualization calculation (#5851)

chore: update to use Number function and use relative import

chore: remove original injection commands (#5840)

* chore: remove injectViaLsp() for measurement, field, and tag values

* chore: remove ExecuteCommandInjectField

* chore: remove ExecuteCommand.InjectField

* chore: remove ExecuteCommandInjectTag and ExecuteCommandInjectTagValue

* chore: remove ExecuteCommandInjectMeasurement

* chore: remove injectViaLsp() implementation

fix: resolve flakiness by filling gaps in call to makeQuartzUseIDPEOrgID() (#5871)
subirjolly pushed a commit that referenced this pull request Sep 23, 2022
chore: rename function

chore: remove unused import

chore: update per review

chore: refactor to remove index out of bounds issue

chore: direct state change without mutator per review

chore: minor changes and add more status code checks to pactch account response

chore: update userAccounts with updated reference

chore: remove not required code

chore: remove response status case which is highly unlikely

chore: remove unnecessary activeAccount check

chore: remove unused import

feat: use quartz, not idpe, for user sessions (#5769)

* fix: use quartz for sessions behind feat flag

* chore: fix relative import of feature flag

* chore: remove unnecessary comment

feat(5340): use default editorText in script editor (#5741)

* feat(5340): use default editorText in script editor; updates to session, new script resource, and clear() methods

* feat(5340): update tests for new default text, and caught edge case for certain flag+action combination

* test: demonstrate that we need the default query set on both the session, and in the script resource. In order to have on-load provide the default message.

* fix: add back the default message, to the default session store

fix(schemaComposition): make highlight to scroll with editor (#5719)

* style: use native monaco editor api for customized style for sync highlight

* chore: run prettier

* chore: add sync off style

* chore: add clickable icon to editor

* chore: update comment

* chore: make sync icon to be a background image and un-clickable

* chore: run prettier

* chore: add comment

* test: update classNames in test

chore: remove avatarWidgetMultiAccountInfo feature flag code (#5803)

* chore: remove avatarWidgetMultiAccountInfo feature flag code

* chore: remove unused imports

Co-authored-by: Subir Jolly <[email protected]>

feat: sync updated account name across header and profile page.

chore: remove unused imports

chore: remove globalheadercontainer

chore: remove not used code

chore: add cloud check fir using useraccountprovider

chore: update to have cloud check for user account provider

chore: add guards to pushing into selected variable array, remove extra reporting (#5791)

* chore: replace not-extensible honeybadger reporting with a console.error to reduce network calls

* chore: remove unused import

* fix: add guard to not push to vari.selected if not extensible

chore(CsvMethod): move context logic into CsvMethod (#5751)

* chore: move context logic into CsvMethod

* fix: prettier

* fix: prettier

* chore: remove csvUploader file

* chore: clean up

chore(5780): track usage of fallback worker (#5782)

feat(idpe-15219): remove pinned items from dashboards (#5789)

feat(idpe-15219): remove pinned items from cypress tests (#5785)

chore: upgrade normalizr to latest (#5792)

chore: remove http-proxy-middleware (#5749)

* chore: upgrade http-proxy-middleware to the latest

* chore: remove http-proxy-middleware

feat(idpe-15219): remove pinnedItems components, no longer used on homepage (#5786)

style: remove the corner radius of the Monaco editor and Results panel (#5772)

* style: remove the border radius for Monaco editor and Results pane

* style: remove margin bottom and update border size for the heading

* style: add top padding for two side bars on the left and right

* style: increase padding to the heading menu

chore: Upgrade chroma-js and it's types to latest (#5798)

* chore: upgrade @types/chroma-js to latest

* chore: upgrade chroma-js to latest

chore: upgrade monaco to latest (#5793)

* chore: upgrade monaco to latest

* chore: change monaco language client to 2.1.0

* chore: change monaco language client to 0.18.1

* chore: tilda for react monaco editor

* chore: yarn lock

chore: upgrade @types/chai to latest (#5797)

fix: page content now visible on mobile devices (#5651)

* fix: page content now visible on mobile devices

* chore: linter

* fix: global header popover

* fix: remove custom style adjustments for dapperscrollbar, dataexplorer, tasks

* fix: remove custom style adjustments for dapperscrollbar, dataexplorer, tasks

Co-authored-by: wdoconnell <[email protected]>

feat(idpe-15219): remove pinned items from Tasks (#5784)

chore: upgrade testing-library to latest (#5795)

* chore: upgrade testing-library to latest

* chore: downgrade testing-library/react to 12
.1.5

fix(sub): password field optional during update (#5812)

feat(idpe-15219): remove pinned items from flows (#5783)

fix(firstMile/arduino): font size for "Note" (#5823)

feat: add multiselect for measurements (#5673)

* feat: add multiselect for measurements

* fix: missed some edgecase behaviors

* fix: updating styling and text per design review

Co-authored-by: drdelambre <drdelambre>

feat(idpe-15219): remove pinnedItems context and notification methods (#5822)

chore: refactor useraccounts to use api layer (#5773)

* chore: refactor useraccounts to use api layer

* chore: rename function

* chore: remove unused import

* chore: update per review

* chore: refactor to remove index out of bounds issue

* chore: direct state change without mutator per review

* chore: minor changes and add more status code checks to pactch account response

* chore: update userAccounts with updated reference

* chore: remove not required code

* chore: remove response status case which is highly unlikely

* chore: remove unnecessary activeAccount check

* chore: remove unused import

* chore: remove unused error notification

* chore: reporting account errors to honebadger

* chore: update payload info for reportErrorThroughHoneyBadger

* chore: update per review

Co-authored-by: Subir Jolly <[email protected]>

chore: upgrade auth0-js to latest (#5796)

chore: fix array length check

chore: use optional chaining for getting length

chore: update name of action function

chore: fix imports path

chore: add account settings page behind cloud

feat: replace isRegionBeta with isUpgradeable from identity (#5808)

chore: update jest-junit (#5827)

fix(firstMile/arduino): scroll to selected bucket (#5825)

chore: upgrade css-loader to v5.2.7 (#5747)

* chore: upgrade css-loader to the latest

* chore: upgrade css-loader to the v5.2.7

* chore: yarn lock

fix: change expected acct endpoint response to single object (#5838)

fix(firstMile): fixing listen data component to listen on component mount (#5830)

* fix(firstMile): fixing listen data component to listen on componentDidMount as well

* fix(firstMile): make a function for reinitializing listening

* fix(firstMile): import

chore: update Giraffe (#5839)

fix(rudderstack): update version to remove app flicker (#5841)

feat: add new Telegraf plugins and update existing plugins (#5842)

feat: add user feedback button (#5831)

* feat: add userpilot feedback button in new script editor

* chore: add feature flag around the feedback button

fix: update to latest giraffe to fix font issues (#5843)

feat: add new time range component for the UI for the new data explorer (#5817)

* wip

* feat: added validation, need to clean up design when they are ready and clean up the component logic

* feat: connect all the things

fix: use react router links in lieu of redirect for gh main menu subelements (#5828)

fix(reporting): guards to prevent accessing null values from useSelector (#5848)

chore: upgrade html2canvas (#5850)

chore: upgrade intersection-observer polyfill (#5853)

fix: if no new cert is provided a user should be able to submit (#5852)

* fix: if no new cert is provided a user should be able to update their sub

* fix: lint

* fix: prettier

chore: upgrade clockface to 6.3.5 (#5854)

chore: remove ability to use quartzMe (#5847)

chore: fix component name

fix: update save script text (#5857)

Co-authored-by: drdelambre <drdelambre>

fix: kick out users if new DE flag is off (#5858)

* fix: let users leave the experiment

* fix: kick them out even more

* fix: force flag to be set on every test

Co-authored-by: drdelambre <drdelambre>

chore(dependencies): update to clockface 6.3.6 to attempt fixing corrupt icon fonts (#5863)

chore: convert string id to number

chore: change currentaccount to account type

chore: update icon for Open and Save (#5860)

fix: add caching to color visualization calculation (#5851)

chore: update to use Number function and use relative import

chore: remove original injection commands (#5840)

* chore: remove injectViaLsp() for measurement, field, and tag values

* chore: remove ExecuteCommandInjectField

* chore: remove ExecuteCommand.InjectField

* chore: remove ExecuteCommandInjectTag and ExecuteCommandInjectTagValue

* chore: remove ExecuteCommandInjectMeasurement

* chore: remove injectViaLsp() implementation

fix: resolve flakiness by filling gaps in call to makeQuartzUseIDPEOrgID() (#5871)
subirjolly pushed a commit that referenced this pull request Sep 23, 2022
chore: rename function

chore: remove unused import

chore: update per review

chore: refactor to remove index out of bounds issue

chore: direct state change without mutator per review

chore: minor changes and add more status code checks to pactch account response

chore: update userAccounts with updated reference

chore: remove not required code

chore: remove response status case which is highly unlikely

chore: remove unnecessary activeAccount check

chore: remove unused import

feat: use quartz, not idpe, for user sessions (#5769)

* fix: use quartz for sessions behind feat flag

* chore: fix relative import of feature flag

* chore: remove unnecessary comment

feat(5340): use default editorText in script editor (#5741)

* feat(5340): use default editorText in script editor; updates to session, new script resource, and clear() methods

* feat(5340): update tests for new default text, and caught edge case for certain flag+action combination

* test: demonstrate that we need the default query set on both the session, and in the script resource. In order to have on-load provide the default message.

* fix: add back the default message, to the default session store

fix(schemaComposition): make highlight to scroll with editor (#5719)

* style: use native monaco editor api for customized style for sync highlight

* chore: run prettier

* chore: add sync off style

* chore: add clickable icon to editor

* chore: update comment

* chore: make sync icon to be a background image and un-clickable

* chore: run prettier

* chore: add comment

* test: update classNames in test

chore: remove avatarWidgetMultiAccountInfo feature flag code (#5803)

* chore: remove avatarWidgetMultiAccountInfo feature flag code

* chore: remove unused imports

Co-authored-by: Subir Jolly <[email protected]>

feat: sync updated account name across header and profile page.

chore: remove unused imports

chore: remove globalheadercontainer

chore: remove not used code

chore: add cloud check fir using useraccountprovider

chore: update to have cloud check for user account provider

chore: add guards to pushing into selected variable array, remove extra reporting (#5791)

* chore: replace not-extensible honeybadger reporting with a console.error to reduce network calls

* chore: remove unused import

* fix: add guard to not push to vari.selected if not extensible

chore(CsvMethod): move context logic into CsvMethod (#5751)

* chore: move context logic into CsvMethod

* fix: prettier

* fix: prettier

* chore: remove csvUploader file

* chore: clean up

chore(5780): track usage of fallback worker (#5782)

feat(idpe-15219): remove pinned items from dashboards (#5789)

feat(idpe-15219): remove pinned items from cypress tests (#5785)

chore: upgrade normalizr to latest (#5792)

chore: remove http-proxy-middleware (#5749)

* chore: upgrade http-proxy-middleware to the latest

* chore: remove http-proxy-middleware

feat(idpe-15219): remove pinnedItems components, no longer used on homepage (#5786)

style: remove the corner radius of the Monaco editor and Results panel (#5772)

* style: remove the border radius for Monaco editor and Results pane

* style: remove margin bottom and update border size for the heading

* style: add top padding for two side bars on the left and right

* style: increase padding to the heading menu

chore: Upgrade chroma-js and it's types to latest (#5798)

* chore: upgrade @types/chroma-js to latest

* chore: upgrade chroma-js to latest

chore: upgrade monaco to latest (#5793)

* chore: upgrade monaco to latest

* chore: change monaco language client to 2.1.0

* chore: change monaco language client to 0.18.1

* chore: tilda for react monaco editor

* chore: yarn lock

chore: upgrade @types/chai to latest (#5797)

fix: page content now visible on mobile devices (#5651)

* fix: page content now visible on mobile devices

* chore: linter

* fix: global header popover

* fix: remove custom style adjustments for dapperscrollbar, dataexplorer, tasks

* fix: remove custom style adjustments for dapperscrollbar, dataexplorer, tasks

Co-authored-by: wdoconnell <[email protected]>

feat(idpe-15219): remove pinned items from Tasks (#5784)

chore: upgrade testing-library to latest (#5795)

* chore: upgrade testing-library to latest

* chore: downgrade testing-library/react to 12
.1.5

fix(sub): password field optional during update (#5812)

feat(idpe-15219): remove pinned items from flows (#5783)

fix(firstMile/arduino): font size for "Note" (#5823)

feat: add multiselect for measurements (#5673)

* feat: add multiselect for measurements

* fix: missed some edgecase behaviors

* fix: updating styling and text per design review

Co-authored-by: drdelambre <drdelambre>

feat(idpe-15219): remove pinnedItems context and notification methods (#5822)

chore: refactor useraccounts to use api layer (#5773)

* chore: refactor useraccounts to use api layer

* chore: rename function

* chore: remove unused import

* chore: update per review

* chore: refactor to remove index out of bounds issue

* chore: direct state change without mutator per review

* chore: minor changes and add more status code checks to pactch account response

* chore: update userAccounts with updated reference

* chore: remove not required code

* chore: remove response status case which is highly unlikely

* chore: remove unnecessary activeAccount check

* chore: remove unused import

* chore: remove unused error notification

* chore: reporting account errors to honebadger

* chore: update payload info for reportErrorThroughHoneyBadger

* chore: update per review

Co-authored-by: Subir Jolly <[email protected]>

chore: upgrade auth0-js to latest (#5796)

chore: fix array length check

chore: use optional chaining for getting length

chore: update name of action function

chore: fix imports path

chore: add account settings page behind cloud

feat: replace isRegionBeta with isUpgradeable from identity (#5808)

chore: update jest-junit (#5827)

fix(firstMile/arduino): scroll to selected bucket (#5825)

chore: upgrade css-loader to v5.2.7 (#5747)

* chore: upgrade css-loader to the latest

* chore: upgrade css-loader to the v5.2.7

* chore: yarn lock

fix: change expected acct endpoint response to single object (#5838)

fix(firstMile): fixing listen data component to listen on component mount (#5830)

* fix(firstMile): fixing listen data component to listen on componentDidMount as well

* fix(firstMile): make a function for reinitializing listening

* fix(firstMile): import

chore: update Giraffe (#5839)

fix(rudderstack): update version to remove app flicker (#5841)

feat: add new Telegraf plugins and update existing plugins (#5842)

feat: add user feedback button (#5831)

* feat: add userpilot feedback button in new script editor

* chore: add feature flag around the feedback button

fix: update to latest giraffe to fix font issues (#5843)

feat: add new time range component for the UI for the new data explorer (#5817)

* wip

* feat: added validation, need to clean up design when they are ready and clean up the component logic

* feat: connect all the things

fix: use react router links in lieu of redirect for gh main menu subelements (#5828)

fix(reporting): guards to prevent accessing null values from useSelector (#5848)

chore: upgrade html2canvas (#5850)

chore: upgrade intersection-observer polyfill (#5853)

fix: if no new cert is provided a user should be able to submit (#5852)

* fix: if no new cert is provided a user should be able to update their sub

* fix: lint

* fix: prettier

chore: upgrade clockface to 6.3.5 (#5854)

chore: remove ability to use quartzMe (#5847)

chore: fix component name

fix: update save script text (#5857)

Co-authored-by: drdelambre <drdelambre>

fix: kick out users if new DE flag is off (#5858)

* fix: let users leave the experiment

* fix: kick them out even more

* fix: force flag to be set on every test

Co-authored-by: drdelambre <drdelambre>

chore(dependencies): update to clockface 6.3.6 to attempt fixing corrupt icon fonts (#5863)

chore: convert string id to number

chore: change currentaccount to account type

chore: update icon for Open and Save (#5860)

fix: add caching to color visualization calculation (#5851)

chore: update to use Number function and use relative import

chore: remove original injection commands (#5840)

* chore: remove injectViaLsp() for measurement, field, and tag values

* chore: remove ExecuteCommandInjectField

* chore: remove ExecuteCommand.InjectField

* chore: remove ExecuteCommandInjectTag and ExecuteCommandInjectTagValue

* chore: remove ExecuteCommandInjectMeasurement

* chore: remove injectViaLsp() implementation

fix: resolve flakiness by filling gaps in call to makeQuartzUseIDPEOrgID() (#5871)
subirjolly added a commit that referenced this pull request Sep 23, 2022
chore: rename function

chore: remove unused import

chore: update per review

chore: refactor to remove index out of bounds issue

chore: direct state change without mutator per review

chore: minor changes and add more status code checks to pactch account response

chore: update userAccounts with updated reference

chore: remove not required code

chore: remove response status case which is highly unlikely

chore: remove unnecessary activeAccount check

chore: remove unused import

feat: use quartz, not idpe, for user sessions (#5769)

* fix: use quartz for sessions behind feat flag

* chore: fix relative import of feature flag

* chore: remove unnecessary comment

feat(5340): use default editorText in script editor (#5741)

* feat(5340): use default editorText in script editor; updates to session, new script resource, and clear() methods

* feat(5340): update tests for new default text, and caught edge case for certain flag+action combination

* test: demonstrate that we need the default query set on both the session, and in the script resource. In order to have on-load provide the default message.

* fix: add back the default message, to the default session store

fix(schemaComposition): make highlight to scroll with editor (#5719)

* style: use native monaco editor api for customized style for sync highlight

* chore: run prettier

* chore: add sync off style

* chore: add clickable icon to editor

* chore: update comment

* chore: make sync icon to be a background image and un-clickable

* chore: run prettier

* chore: add comment

* test: update classNames in test

chore: remove avatarWidgetMultiAccountInfo feature flag code (#5803)

* chore: remove avatarWidgetMultiAccountInfo feature flag code

* chore: remove unused imports

Co-authored-by: Subir Jolly <[email protected]>

feat: sync updated account name across header and profile page.

chore: remove unused imports

chore: remove globalheadercontainer

chore: remove not used code

chore: add cloud check fir using useraccountprovider

chore: update to have cloud check for user account provider

chore: add guards to pushing into selected variable array, remove extra reporting (#5791)

* chore: replace not-extensible honeybadger reporting with a console.error to reduce network calls

* chore: remove unused import

* fix: add guard to not push to vari.selected if not extensible

chore(CsvMethod): move context logic into CsvMethod (#5751)

* chore: move context logic into CsvMethod

* fix: prettier

* fix: prettier

* chore: remove csvUploader file

* chore: clean up

chore(5780): track usage of fallback worker (#5782)

feat(idpe-15219): remove pinned items from dashboards (#5789)

feat(idpe-15219): remove pinned items from cypress tests (#5785)

chore: upgrade normalizr to latest (#5792)

chore: remove http-proxy-middleware (#5749)

* chore: upgrade http-proxy-middleware to the latest

* chore: remove http-proxy-middleware

feat(idpe-15219): remove pinnedItems components, no longer used on homepage (#5786)

style: remove the corner radius of the Monaco editor and Results panel (#5772)

* style: remove the border radius for Monaco editor and Results pane

* style: remove margin bottom and update border size for the heading

* style: add top padding for two side bars on the left and right

* style: increase padding to the heading menu

chore: Upgrade chroma-js and it's types to latest (#5798)

* chore: upgrade @types/chroma-js to latest

* chore: upgrade chroma-js to latest

chore: upgrade monaco to latest (#5793)

* chore: upgrade monaco to latest

* chore: change monaco language client to 2.1.0

* chore: change monaco language client to 0.18.1

* chore: tilda for react monaco editor

* chore: yarn lock

chore: upgrade @types/chai to latest (#5797)

fix: page content now visible on mobile devices (#5651)

* fix: page content now visible on mobile devices

* chore: linter

* fix: global header popover

* fix: remove custom style adjustments for dapperscrollbar, dataexplorer, tasks

* fix: remove custom style adjustments for dapperscrollbar, dataexplorer, tasks

Co-authored-by: wdoconnell <[email protected]>

feat(idpe-15219): remove pinned items from Tasks (#5784)

chore: upgrade testing-library to latest (#5795)

* chore: upgrade testing-library to latest

* chore: downgrade testing-library/react to 12
.1.5

fix(sub): password field optional during update (#5812)

feat(idpe-15219): remove pinned items from flows (#5783)

fix(firstMile/arduino): font size for "Note" (#5823)

feat: add multiselect for measurements (#5673)

* feat: add multiselect for measurements

* fix: missed some edgecase behaviors

* fix: updating styling and text per design review

Co-authored-by: drdelambre <drdelambre>

feat(idpe-15219): remove pinnedItems context and notification methods (#5822)

chore: refactor useraccounts to use api layer (#5773)

* chore: refactor useraccounts to use api layer

* chore: rename function

* chore: remove unused import

* chore: update per review

* chore: refactor to remove index out of bounds issue

* chore: direct state change without mutator per review

* chore: minor changes and add more status code checks to pactch account response

* chore: update userAccounts with updated reference

* chore: remove not required code

* chore: remove response status case which is highly unlikely

* chore: remove unnecessary activeAccount check

* chore: remove unused import

* chore: remove unused error notification

* chore: reporting account errors to honebadger

* chore: update payload info for reportErrorThroughHoneyBadger

* chore: update per review

Co-authored-by: Subir Jolly <[email protected]>

chore: upgrade auth0-js to latest (#5796)

chore: fix array length check

chore: use optional chaining for getting length

chore: update name of action function

chore: fix imports path

chore: add account settings page behind cloud

feat: replace isRegionBeta with isUpgradeable from identity (#5808)

chore: update jest-junit (#5827)

fix(firstMile/arduino): scroll to selected bucket (#5825)

chore: upgrade css-loader to v5.2.7 (#5747)

* chore: upgrade css-loader to the latest

* chore: upgrade css-loader to the v5.2.7

* chore: yarn lock

fix: change expected acct endpoint response to single object (#5838)

fix(firstMile): fixing listen data component to listen on component mount (#5830)

* fix(firstMile): fixing listen data component to listen on componentDidMount as well

* fix(firstMile): make a function for reinitializing listening

* fix(firstMile): import

chore: update Giraffe (#5839)

fix(rudderstack): update version to remove app flicker (#5841)

feat: add new Telegraf plugins and update existing plugins (#5842)

feat: add user feedback button (#5831)

* feat: add userpilot feedback button in new script editor

* chore: add feature flag around the feedback button

fix: update to latest giraffe to fix font issues (#5843)

feat: add new time range component for the UI for the new data explorer (#5817)

* wip

* feat: added validation, need to clean up design when they are ready and clean up the component logic

* feat: connect all the things

fix: use react router links in lieu of redirect for gh main menu subelements (#5828)

fix(reporting): guards to prevent accessing null values from useSelector (#5848)

chore: upgrade html2canvas (#5850)

chore: upgrade intersection-observer polyfill (#5853)

fix: if no new cert is provided a user should be able to submit (#5852)

* fix: if no new cert is provided a user should be able to update their sub

* fix: lint

* fix: prettier

chore: upgrade clockface to 6.3.5 (#5854)

chore: remove ability to use quartzMe (#5847)

chore: fix component name

fix: update save script text (#5857)

Co-authored-by: drdelambre <drdelambre>

fix: kick out users if new DE flag is off (#5858)

* fix: let users leave the experiment

* fix: kick them out even more

* fix: force flag to be set on every test

Co-authored-by: drdelambre <drdelambre>

chore(dependencies): update to clockface 6.3.6 to attempt fixing corrupt icon fonts (#5863)

chore: convert string id to number

chore: change currentaccount to account type

chore: update icon for Open and Save (#5860)

fix: add caching to color visualization calculation (#5851)

chore: update to use Number function and use relative import

chore: remove original injection commands (#5840)

* chore: remove injectViaLsp() for measurement, field, and tag values

* chore: remove ExecuteCommandInjectField

* chore: remove ExecuteCommand.InjectField

* chore: remove ExecuteCommandInjectTag and ExecuteCommandInjectTagValue

* chore: remove ExecuteCommandInjectMeasurement

* chore: remove injectViaLsp() implementation

fix: resolve flakiness by filling gaps in call to makeQuartzUseIDPEOrgID() (#5871)

Co-authored-by: Subir Jolly <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants