search

akosbalasko / yarle

Yarle - The ultimate converter of Evernote notes to Markdown
https://github.com/akosbalasko/yarle
MIT License
1.4k stars 82 forks source link

Timestamps of MD files are being set to conversion time #623

Closed lawabider closed 5 months ago

lawabider commented 5 months ago

I'm exporting from Evernote 6.25.1.9091 , in Windows.

github-actions[bot] commented 5 months ago

Yihaa, thank you for reporting me this issue and to let me improve Yarle!

akosbalasko commented 5 months ago

Hi @lawabider , There are two options here: I'm not sure that the exported enex file contains the creation timestamp if the export was done using this old version of Evernote. The other point would be Windows, because Yarle uses a 3rd party lib to handle creation times dedicatedly for Windows (I mean to be transparent by platform). Which Windows version do you use?

lawabider commented 5 months ago

My Windows version is 10.0.22000. I think I was wrong about the creation timestamp; it was reset when I copied the files. I should have moved them instead. I still don't think that the file modified timestamp matches the note's last updated timestamp. I have been assuming that it's supposed to.

ipcjs commented 5 months ago

The PR has fixed the issue: #624

lawabider commented 5 months ago

Can someone help me to use npx with github urls? This works npx -p yarle-evernote-to-md@latest yarle --configFile config.json

But this attempt failed: npx -p github:akosbalasko/yarle@latest yarle --configFile config.json

akosbalasko commented 5 months ago

hi @lawabider Why do you want to executed yarle dedicatedly from github? the latest version in npm is what you see here on github, so it should be the same.

akosbalasko commented 5 months ago

@ipcjs please check your PR, it fails on Ubuntu and Mac, looks working on Windows only.

lawabider commented 5 months ago

@akosbalasko I was hoping to try ipcjs's fork. I'm in a bit of a rabbit hole here, being weak at github and knowing nothing of the javascript world.

akosbalasko commented 5 months ago

@lawabider that's a bit more complicated, I mean using an other fork, you'd need to build his fork etc... so I would suggest to wait a bit until his PR is approved and merged, currently we are facing with some platform issues (works on Windows, but do not work on any other OS). I try to resolve it today or at most tomorrow.

ipcjs commented 5 months ago

@lawabider I have fixed the test cases

akosbalasko commented 5 months ago

@ipcjs thanks for the contribution, merged and released!