-
Notifications
You must be signed in to change notification settings - Fork 4
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
Cockpit next? #6
Comments
Due to my current workload, not any time soon. The thing is that current module version uses heavily angular which has been dropped in cockpit-next in favor of Riot.js which I have no experience with. As cockpit requires modern browser to run, some angular utilities could be moved native (Promise, XHR2) I'll keep this issue open until I'll be able to tell more. |
@piotr-cz I started to work on an import functionality in cockpit next, feel invited to join ;-) |
I'd feel more comfortable if Import functionality would be a separate and independent module. |
I disagree - I think the core integration makes perfect sense how Artur On Wed, Aug 24, 2016, 6:08 AM Piotr notifications@github.com wrote:
|
In my opinion it makes sense to have built-in import functionality for JSON files previously exported from Cockpit-next or Cockpit where data structure is very similar. But for me personally main goal for this module is to import CSV/ spreadsheet data, that I receive from customer, with for example gelocating addresses. |
any plans to update your add-on for the new version?
The text was updated successfully, but these errors were encountered: