fergiemcdowall / norch

A search server that can be installed with npm

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

check if /snapshot or /latestSnapshot is the issue

eklem opened this issue · comments

Install the pullrequest-version and test. Reference in documentation talks about /snapshot, but the actual documentation shows it as /createSnapshot..

Try to post on both...

A little investigation on the error on this pull request.

/createSnapshot should be called /snapshot in the documentation, or we should change the code + test. What do you think, @fergiemcdowall ? Not the cause of the error in the pull request though.

ToDo: Fire up Norch, try to create a snapshot, check the disk to see if folder + files are created.

I wonder if latestSnapshot should be removed from the core. Its probably enough that /snapshot works?

Yes, we can add something like that if we get an actual use case.

Hmm, we have

  • snapshot (create a snapshot)
  • latestSnapshot
  • listSnapshots