Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

feat(encodable): add function for setting domain #256

Merged
merged 5 commits into from
Nov 15, 2019
Merged

Conversation

kristw
Copy link
Contributor

@kristw kristw commented Nov 15, 2019

馃弳 Enhancements

Add helper functions for setting domain

@kristw kristw requested a review from a team as a code owner November 15, 2019 02:11
@netlify
Copy link

netlify bot commented Nov 15, 2019

Deploy preview for superset-ui ready!

Built with commit c007e9c

https://deploy-preview-256--superset-ui.netlify.com

@codecov
Copy link

codecov bot commented Nov 15, 2019

Codecov Report

Merging #256 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #256   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files         148    149    +1     
  Lines        1636   1644    +8     
  Branches      434    435    +1     
=====================================
+ Hits         1636   1644    +8
Impacted Files Coverage 螖
...rset-ui-encodable/src/parsers/scale/applyDomain.ts 100% <100%> (酶) 猬嗭笍
...t-ui-encodable/src/utils/removeUndefinedAndNull.ts 100% <100%> (酶)
...perset-ui-encodable/src/encoders/ChannelEncoder.ts 100% <100%> (酶) 猬嗭笍

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 2cf401b...c007e9c. Read the comment docs.

@kristw kristw merged commit ece5bab into master Nov 15, 2019
@delete-merged-branch delete-merged-branch bot deleted the kristw--scale-helper branch November 15, 2019 19:43
kristw pushed a commit that referenced this pull request Apr 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant