Skip to content
This repository was archived by the owner on Feb 2, 2025. It is now read-only.

Add a service to provide the DataTable instances #174

Closed
l-lin opened this issue Jan 10, 2015 · 0 comments
Closed

Add a service to provide the DataTable instances #174

l-lin opened this issue Jan 10, 2015 · 0 comments
Milestone

Comments

@l-lin
Copy link
Owner

l-lin commented Jan 10, 2015

Instead of relying on an event (like catching the event event:dataTableLoaded), it's better to use a service that provides the list of the DataTables that are instanciated.
This feature implies:

  • the creation of the service that will provide an array of DT instances along with the directive instance
  • the removal of the event event:dataTableLoaded
  • the clumsy API dtOptions.reload() will be moved to the directive instance. Indeed, reloading data should not be on the options of the DT. It should rather be within the method of the instance of the directive.
  • open opportunities to add new APIs directly to the directive instance
@l-lin l-lin added this to the v0.4.0 milestone Jan 10, 2015
@l-lin l-lin closed this as completed Jan 10, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant