-
Notifications
You must be signed in to change notification settings - Fork 1
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
Refactor app branch - never got this to work. #1
Conversation
…d fixed the edit notebook and add new notebook (almost completely working)
… get the way this is supposed to actually do things. woot!!! need to do: - fix tags up nice, - fix the NoteNav so counts update correctly for notebooks and tags
…s to work without the ability to store arrays tho huh doods?
…ting for edit notes working finally. need to make delete and the css of the tag field better
… delete tags finally done!
…ings, so for now i am putting in a new branch cuz its all broken now.
…being thrown away. as per usual. i suck
* @param {*} note | ||
* @param {*} allNoteTags | ||
*/ | ||
export function editNoteTags(note, tags) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could be useful stuff here
dispatch(addNoteRejectedAction()); | ||
}); | ||
} | ||
return dispatch => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could be useful here
* @param {*} note | ||
* @param {*} callback | ||
*/ | ||
export function editTags(tags, note) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
useful
* @param {*} notebook | ||
*/ | ||
export function getNotebookCount(notes, notebook) { | ||
let count = 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need all of this stuff
Putting PRs up for the branches I worked on, but dont intend to merge or use so I can view the code that I wrote for them.
With this branch I learned a large amount around working with firebase and how to keep track of unique ids in a normalized data structure. I'd like to reference the code easily, so putting it here.
Also, I did write a good amount of useful functions and other things for this which could be useful...