Skip to content
This repository has been archived by the owner on Jan 9, 2020. It is now read-only.

Frontend - refactor #14

Closed
3 of 5 tasks
mtbarta opened this issue Jun 22, 2018 · 2 comments
Closed
3 of 5 tasks

Frontend - refactor #14

mtbarta opened this issue Jun 22, 2018 · 2 comments
Projects

Comments

@mtbarta
Copy link
Owner

mtbarta commented Jun 22, 2018

I'm in the process of adding unit tests. This doesn't, however, help the fact that the code is a mess.

  • I need to make a lot of note functionality into mixins. I can't really think of a great way to solve my inheritance issue otherwise.

  • GraphQL statements are in their own folders for the notebook, but not for the note menu queries.

  • get around the NoteWrapper abstraction?

  • Flatten out the notebook directory

  • Add tslint and rewrite components into TS

@mtbarta
Copy link
Owner Author

mtbarta commented Jun 22, 2018

one of the issues i struggle with is event passing in the context of the DOM.

https://alligator.io/vuejs/global-event-bus/ might be a good path forward to alleviate this and clean up the front end.

@mtbarta
Copy link
Owner Author

mtbarta commented Jun 22, 2018

https://alligator.io/vuejs/linting-vue-typescript-wotan/

I'm not sure why all of this is necessary to lint typescript in vue. I have no perspective on how other packages lint in vue.

@mtbarta mtbarta added this to Required in Release 1.0 Jun 23, 2018
@mtbarta mtbarta closed this as completed Nov 24, 2018
Release 1.0 automation moved this from Required to Done Nov 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Release 1.0
  
Done
Development

No branches or pull requests

1 participant