-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Add attachment to note #1825
Comments
I think most of your concerns sound right.
I agree with you. We should use
Could you describe more about this? And, I'm really sorry that there are lots of crap in the code of Boostnote. The reasons I couldn't catch up the current code are:
Yea, we should refactor them and make one reliable definition.
TBH, we've been preparing an app, which probably replace the current Boostnote app. So, lots of things will be written from the scratch and it will use Typescript. |
@Rokt33r can you already tell more about this new app? will it be open source and free as well? |
@Rokt33r btw: my issue wasn't meant as a accusation. I can imagine that it's hard to maintain such a big project. I hope to make the project a little better (as it's intended by the open source idea) with my constribution. I've only mentioned the issues above so that's possible to understand why I try to refactor a few things. I hope to finish my fix during the comming weekend. I'm looking forward to hearing your thoughts about it. |
@ehhc Yes, it will be opensource and free! |
Fixes #1825 Refactoring of the attachment/image management
In #1723 i asked for the feature to include attachments in nodes. I've submitted a really ugly quick fix. Unfortunately there are still some points:
My quick fix hasn’t fix these issues.
I’m working on them at the moment and will submit a more suitable solution as soon as possible.
The text was updated successfully, but these errors were encountered: