-
Notifications
You must be signed in to change notification settings - Fork 20
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
Migrate away from AngularJS #74
Comments
Sounds good Is this production ready ? |
It is just one possibility, there is an Angular based version too or even a PHP server: https://github.com/flowjs/ngx-flow Best option would be to use only flow.js without any dependencies like Angular and AngularJS as to minimize the amount of problems relying on 3rd party libraries altogether: |
Do you mean vue.js + flow.js (not the old currently used version) , but no ng-flow or ngx-flow wrapper ? |
Pretty much yes, no Angular or AngularJS dependencies at all. Vue.js is pretty much the standard now for Nextcloud apps anyway. |
Because we no longer use ng-flow when #73 gets merged it should be pretty easy to migrate to vue.js |
The "bug" label should probably be removed from this issue. |
Fixed by #73. |
AngularJS is in LTS mode currently and will be deprecated in 2021 after a 3 year period. We should move away from AngularJS to Vue probably at some point in the future.
One possible library would be to use vue-flow as an alternative:
https://github.com/shenron/vue-flow
The text was updated successfully, but these errors were encountered: