Skip to content
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

create note with selected tags #2498

Merged
merged 4 commits into from
Nov 25, 2018
Merged

Conversation

daiyam
Copy link
Contributor

@daiyam daiyam commented Oct 15, 2018

This change creates a note with the tags selected in the SideNav.

@ZeroX-DG ZeroX-DG added the awaiting review ❇️ Pull request is awaiting a review. label Oct 15, 2018
@ZeroX-DG
Copy link
Member

Should we add a checkbox to notify and allow users to not create a new note with selected tags?

@daiyam
Copy link
Contributor Author

daiyam commented Oct 23, 2018

I could add an option so that new notes use the selected tags or not. Would it be ok?

@ZeroX-DG
Copy link
Member

Yeah @daiyam that sounds good!

@daiyam
Copy link
Contributor Author

daiyam commented Nov 8, 2018

here we go 😉

@ZeroX-DG
Copy link
Member

Hey @daiyam is it me or your code causing this problem?
bb

@daiyam
Copy link
Contributor Author

daiyam commented Nov 11, 2018

@ZeroX-DG sadly, no. The issue is on master...

If you add console.log(location.pathname) at the top of the method tagListComponent(line 209) in the file /Boostnote/browser/main/SideNav/index.js, when you select the second tag, you will see in the console:

/tags/now hello
/tags/now%20hello
/tags/now%20hello
/tags/now%20hello

Beside the fact that the list is updated 4 times for 1 click, you can see that the spaces have been replaced by %20, so the code can't get individual tags (it splits on space...).

It happens since the merge Merge pull request #2484 from crakoucas/fix-electron-glibc-segmentation-fault (106caf2).
Maybe due to the newer electron...

@daiyam
Copy link
Contributor Author

daiyam commented Nov 11, 2018

@ZeroX-DG I've made the PR #2600 to fix the issue on master.

Copy link
Member

@ZeroX-DG ZeroX-DG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🎉

@ZeroX-DG ZeroX-DG added approved 👍 Pull request has been approved by sufficient reviewers. and removed awaiting review ❇️ Pull request is awaiting a review. labels Nov 15, 2018
@Rokt33r Rokt33r added next release (v0.11.12) and removed approved 👍 Pull request has been approved by sufficient reviewers. labels Nov 25, 2018
@Rokt33r Rokt33r merged commit b021bb7 into BoostIO:master Nov 25, 2018
@daiyam daiyam deleted the create-note-with-tags branch November 25, 2018 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants