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

Change the :storage* links to attachments #2937

Closed
wants to merge 3 commits into from
Closed

Change the :storage* links to attachments #2937

wants to merge 3 commits into from

Conversation

CedricHerzog
Copy link

Description

The HTML export creates a folder "attachements" with all the files linked to the Boostnote .md file.
However the links created in the .html file do not point to this "attachements" folder and keep using ":storage/.." representation as used in the Boostnote .md file.

This request changes the links in the HTML export from :storage/12323-2133123-21312323-123323/file.extension to attachments/file.extension.

Works on windows if folders separated by "%".

I uses a simple regex :storage(\/|%)[a-zA-Z0-9-]*and replaces it by "attachements".

Issue fixed

Type of changes

  • ⚪ Bug fix (Change that fixed an issue)
  • ⚪ Breaking change (Change that can cause existing functionality to change)
  • ⚪ Improvement (Change that improves the code. Maybe performance or development improvement)
  • 🔘 Feature (Change that adds new functionality)
  • ⚪ Documentation change (Change that modifies documentation. Maybe typo fixes)

Checklist:

  • 🔘 My code follows the project code style
  • ⚪ I have written test for my code and it has been tested
  • 🔘 All existing tests have been passed
  • 🔘 I have attached a screenshot/video to visualize my change if possible

Screenshot 2019-03-20 at 00 07 19

@ZeroX-DG ZeroX-DG added the awaiting review ❇️ Pull request is awaiting a review. label Mar 20, 2019
@ZeroX-DG
Copy link
Member

@CedricHerzog can you fix the conflict?

@dredav
Copy link
Contributor

dredav commented Mar 21, 2019

Duplicate of fix #2901 or #2897?

@CedricHerzog
Copy link
Author

Duplicate of fix #2901 or #2897?

As I understand the goal of these fixes was to fix the creation of the "attachments" folder during the export. My fix's purpose is to set the right paths to these folder in the HTML file.

@CedricHerzog
Copy link
Author

Any news?

@incredibleweirdo
Copy link

Duplicate of fix #2901 or #2897?

As I understand the goal of these fixes was to fix the creation of the "attachments" folder during the export. My fix's purpose is to set the right paths to these folder in the HTML file.

@CedricHerzog This is a duplicate of #2897 to be sure. It's implemented differently, yes, but they are addressing the same issue.

@noklam
Copy link

noklam commented Nov 8, 2019

any update on this? this broken link is very annoying when I want to export a few folder of markdown.. Would appreciate if this can be fixed.

@noklam
Copy link

noklam commented Nov 10, 2019

any update on this? this broken link is very annoying when I want to export a few folder of markdown.. Would appreciate if this can be fixed.

As a quick fix to export markdown for other usage, I use this script to copy the storage and convert the notes to markdown.

https://gist.github.com/noklam/08d7afd25dd6e15b0208c3b782ebebfc

@Flexo013
Copy link
Contributor

The main reviewers are currently very busy so it might take some time for PRs to be reviewed.

@CedricHerzog
Copy link
Author

The main reviewers are currently very busy so it might take some time for PRs to be reviewed.

Some time OK, but 8 months is a bit long.
Please close the pull request if it really is a duplicate as @incredibleweirdo explained.

@ZeroX-DG
Copy link
Member

Close because it's a duplicate of #2897

@ZeroX-DG ZeroX-DG closed this Mar 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review ❇️ Pull request is awaiting a review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants