doorstop icon indicating copy to clipboard operation
doorstop copied to clipboard

Add additional server features

Open jacebrowning opened this issue 10 years ago • 7 comments

TODO:

  • [x] wait for #117
  • [ ] make prefixes and UIDs clickable
  • [ ] add a temporary document file to store the last number
  • [ ] periodically pull from the "origin" (add --no-pull to disable this)
  • [ ] add an option to hide all web UI elements (--hidden)
  • [ ] determine if requiring a server should be the default

jacebrowning avatar Aug 09 '14 23:08 jacebrowning

I'm adding some basic navigation so you can click on prefixes and UIDs at the moment.

jmacarthur avatar Mar 30 '17 15:03 jmacarthur

Sounds good. I'm not sure how many of the items on this checklist are still desired by anyone, but clickable links would be great!

jacebrowning avatar Mar 30 '17 16:03 jacebrowning

I've also added some features in the branch feature/serverjs . If you start a server and navigate to a document it will be displayed with edit buttons that let you edit the text of an item,

michaelnt avatar Mar 30 '17 18:03 michaelnt

@jmacarthur @michaelnt

Do you have any thoughts on spitting out the server to it's own project?

Advantages include:

  • Letting some one else manage releases at a faster pace than the core code
  • Removing potential web security vulnerabilities from the core code
  • Structuring the server project to be web-focused in terms of tooling and testing

jacebrowning avatar Mar 30 '17 19:03 jacebrowning

I'm not planning to make massive changes and I'm not in a rush to bundle this into a release yet. I'm happy enough to keep plugging away at this branch. I think the changes to the server/main.py can be tested with the existing tools.

The server doesn't support any authentication and should only be run on localhost or in a trusted environment I guess that branch does add the ability to modify an item that wasn't there previously but someone then has to commit that change and the changes are versioned.

I guess it might be a pain to integrate javascript unit testing into this repo so that is one argument for putting the server.html somewhere else.

I'd prefer to leave it here where there is more change of someone helping.

michaelnt avatar Mar 30 '17 20:03 michaelnt

The server doesn't support any authentication and should only be run on localhost or in a trusted environment

👍

Maybe server/main.py will eventually inspire someone to make a fully-fledged Doorstop web UI to compete with some of the other offerings out there. 😄

jacebrowning avatar Mar 30 '17 20:03 jacebrowning

There's some code and templates which are common between the bottle server and the static HTML publisher and I'd quite like to keep it that way.

jmacarthur avatar Mar 31 '17 09:03 jmacarthur