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

Add types for the data-table components #498

Closed
wants to merge 6 commits into from
Closed

Conversation

Windvis
Copy link
Contributor

@Windvis Windvis commented Jul 11, 2024

This copies over the ember-data-table .js code so we no longer need the addon itself. We were already overriding all the templates. This will make things easier to maintain and improve in the future.

Linting errors are silenced with inline ignore comments. They will be fixed once we convert the code to .gts.

This is a good base to start the .gts conversion from.

TODO:

  • Copy over the tests (and tweak them to match our version of the components)
  • Add types for the classic components

Out of scope:

  • Convert the components to .gts
  • deprecate things that we consider private and don't want apps to use directly

package.json Show resolved Hide resolved
@Windvis

This comment was marked as outdated.

@Windvis Windvis mentioned this pull request Nov 5, 2024
This copies over the ember-data-table .js code so we no longer need the addon itself. We were already overriding all the templates. This will make things easier to maintain and improve in the future.

Linting errors are silenced with inline ignore comments. They will be fixed once we convert the code to .gts.
Used by the addon components, but ember-data-table was bringing it in. Can be removed once we refactor the code.
It seems we were still depending on the one from the ember-data-table addon.
It seems we accidentally prefixed a block variable with `this` when resolving some deprecation warnings.
This copies over the data table tests that aren't empty and does the bare minimum to make them pass with our custom templates.
@Windvis Windvis force-pushed the types/au-data-table branch from b21b14e to 2268219 Compare November 7, 2024 09:17
@Windvis Windvis added internal Used for internal changes that still require a mention in the changelog/release notes. enhancement Used when the PR adds a new feature or enhancement. and removed internal Used for internal changes that still require a mention in the changelog/release notes. labels Nov 7, 2024
@Windvis Windvis changed the title Inline the ember-data-table components Add types for the data-table components Nov 7, 2024
@Windvis
Copy link
Contributor Author

Windvis commented Nov 12, 2024

Closing since it's essentially the same as #525. I'll reopen it once I start on the actual type work.

@Windvis Windvis closed this Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Used when the PR adds a new feature or enhancement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant