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

[Feature] Table Generic Typing #818

Merged
merged 2 commits into from
Aug 13, 2021
Merged

Conversation

grahamhency
Copy link
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows our guidelines
    Tests for the changes have been added
  • Docs have been added or updated

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Style Update (CSS)
  • Other... Please describe:

What is the current behavior?

Resolves #813

What is the new behavior?

You can specify a type generic for the GoTableConfig interface now.

Does this PR introduce a breaking change?

  • Yes
  • No

@grahamhency grahamhency added feature New feature request in progress Work on this issue is in progress labels Aug 12, 2021
Copy link
Contributor

@jiaweicai92 jiaweicai92 left a comment

Choose a reason for hiding this comment

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

The rest looks good to me.

Copy link
Contributor

@jiaweicai92 jiaweicai92 left a comment

Choose a reason for hiding this comment

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

Looks good to me👍

@grahamhency grahamhency merged commit 0de618a into dev Aug 13, 2021
@grahamhency grahamhency deleted the feature_table_generic_typing branch August 13, 2021 03:34
@grahamhency grahamhency mentioned this pull request Aug 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature request in progress Work on this issue is in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Add ability to pass a type to the GoTableConfig for the tableData type
2 participants