fergiemcdowall/norch

check if /snapshot or /latestSnapshot is the issue

eklem opened this issue · 5 comments

eklem commented

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.

eklem commented

/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.

eklem commented

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?

eklem commented

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

eklem commented

Hmm, we have

  • snapshot (create a snapshot)
  • latestSnapshot
  • listSnapshots