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: UI toast typo #13026

Merged
merged 3 commits into from
Feb 10, 2021
Merged

fix: UI toast typo #13026

merged 3 commits into from
Feb 10, 2021

Conversation

dpgaspar
Copy link
Member

@dpgaspar dpgaspar commented Feb 9, 2021

SUMMARY

Fixes small typo and message toast handling

Before:
Screenshot 2021-02-09 at 15 43 22

After:
Screenshot 2021-02-09 at 15 42 17

ADDITIONAL INFORMATION

  • Has associated issue:
  • Changes UI
  • Requires DB Migration.
  • Confirm DB Migration upgrade and downgrade tested.
  • Introduces new feature or API
  • Removes existing feature or API

`There was an issue fetching your recent acitivity: ${e}`,
`There was an issue fetching your recent activity: ${e}`,
Copy link
Member

Choose a reason for hiding this comment

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

Let's translate this

Copy link
Member

@villebro villebro left a comment

Choose a reason for hiding this comment

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

LGTM. Not sure what the convention on super short variable names, but while we're at it, perhaps we could also replace e with errMsg or similar that's used in many other places in the code base.

@codecov-io
Copy link

codecov-io commented Feb 9, 2021

Codecov Report

Merging #13026 (33679c3) into master (50fa100) will increase coverage by 20.32%.
The diff coverage is 92.01%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master   #13026       +/-   ##
===========================================
+ Coverage   52.64%   72.97%   +20.32%     
===========================================
  Files         480      546       +66     
  Lines       17300    20175     +2875     
  Branches     4481     5268      +787     
===========================================
+ Hits         9108    14723     +5615     
+ Misses       8192     5327     -2865     
- Partials        0      125      +125     
Flag Coverage Δ
cypress 53.38% <65.27%> (+0.73%) ⬆️
javascript 61.86% <65.15%> (?)

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

Impacted Files Coverage Δ
...et-frontend/src/dashboard/actions/nativeFilters.ts 82.60% <ø> (-1.61%) ⬇️
...tend/src/dashboard/components/DashboardBuilder.jsx 85.89% <ø> (ø)
...src/dashboard/components/FiltersBadge/selectors.ts 91.66% <ø> (+1.66%) ⬆️
...ters/FilterConfigModal/CancelConfirmationAlert.tsx 100.00% <ø> (ø)
...s/nativeFilters/FilterConfigModal/ColumnSelect.tsx 77.77% <ø> (ø)
...ts/nativeFilters/FilterConfigModal/FiltersList.tsx 0.00% <0.00%> (ø)
superset-frontend/src/dashboard/reducers/types.ts 0.00% <ø> (ø)
...src/dashboard/util/activeDashboardNativeFilters.ts 96.42% <ø> (+7.53%) ⬆️
...shboard/util/charts/getFormDataWithExtraFilters.ts 93.10% <ø> (+1.10%) ⬆️
...perset-frontend/src/views/CRUD/welcome/Welcome.tsx 88.88% <ø> (+85.94%) ⬆️
... and 432 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eb5dc38...33679c3. Read the comment docs.

@pull-request-size pull-request-size bot added size/M and removed size/XS labels Feb 9, 2021
@dpgaspar dpgaspar merged commit 3e0681b into apache:master Feb 10, 2021
@dpgaspar dpgaspar deleted the fix/typo branch February 10, 2021 08:17
amitmiran137 pushed a commit to simcha90/incubator-superset that referenced this pull request Feb 10, 2021
* master:
  fix: UI toast typo (apache#13026)
  feat(db engines): add support for Opendistro Elasticsearch (AWS ES) (apache#12602)
  fix(build): black failing on master, add to required checks (apache#13039)
  fix: time filter db migration optimization (apache#13015)
  fix: untranslated text content of Dashboard page (apache#13024)
  fix(ci): remove signature requirements for commits to master (apache#13034)
  fix: add alerts and report to default config (apache#12999)
  docs(changelog): add entries for 1.0.1 (apache#12981)
  ci: skip cypress if no code changes (apache#12982)
  chore: add cypress required checks for branch protection (apache#12970)
  Refresh dashboard list after bulk delete (apache#12945)
  Updates storybook to version 6.1.17 (apache#13014)
  feat: Save datapanel state in local storage (apache#12996)
  fix: added text and changed margins (apache#12923)
  chore: Swap Slack Url 2 more places (apache#13004)
amitmiran137 pushed a commit to nielsen-oss/superset that referenced this pull request Feb 14, 2021
* fix: UI toast typo

* translate

* fix error msg
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 1.2.0 labels Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels preset-io size/M 🚢 1.2.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants