Skip to content
This repository has been archived by the owner on Jun 25, 2020. It is now read-only.

feat: add Canada to Country Maps #48

Merged
merged 4 commits into from
Apr 17, 2019
Merged

feat: add Canada to Country Maps #48

merged 4 commits into from
Apr 17, 2019

Conversation

andy-clapson
Copy link
Contributor

🏆 Enhancements - adding Canada onto the ui plugins options.

@andy-clapson andy-clapson requested a review from a team as a code owner April 10, 2019 14:41
@ghost
Copy link

ghost commented Apr 10, 2019

There were the following issues with this Pull Request

  • Commit: a754330
    • ✖ message may not be empty
    • ✖ type may not be empty

You may need to change the commit messages to comply with the repository contributing guidelines.


🤖 This comment was generated by commitlint[bot]. Please report issues here.

Happy coding!

@ghost
Copy link

ghost commented Apr 10, 2019

There were the following issues with this Pull Request

  • Commit: a754330
    • ✖ message may not be empty
    • ✖ type may not be empty
  • Commit: c8e17e3
    • ✖ message may not be empty
    • ✖ type may not be empty

You may need to change the commit messages to comply with the repository contributing guidelines.


🤖 This comment was generated by commitlint[bot]. Please report issues here.

Happy coding!

@kristw
Copy link
Collaborator

kristw commented Apr 12, 2019

Would you mind rebasing on top of master?

@kristw kristw changed the title Adding Canada to Country Maps feat: add Canada to Country Maps Apr 12, 2019
@andy-clapson
Copy link
Contributor Author

Would you mind rebasing on top of master?

Of course! Sorry about that.

@kristw kristw merged commit eb9ffb5 into apache-superset:master Apr 17, 2019
nytai pushed a commit to preset-io/superset-ui-plugins that referenced this pull request Apr 27, 2020
…superset#48)

* migrate query code from incubator-superset

* Getting tests to pass

- Up build-config version to pick up a fix to eslint-typescript-parser
- Remove usage of default exports in favor of named exports unless the export is the only thing being exported out of a module
- Fixing up a few linting errors

* - Remove DatasourceKey interface in favor of readonly id and type properties on the DatasourceKey class directly.
- Adding tests for DatasourceKey.
nytai pushed a commit to preset-io/superset-ui-plugins that referenced this pull request Apr 27, 2020
…rset#48)

travis build failed due to another issue, which is fixed in master.
verified storybook and the new map is working correctly.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants