[feature request] [bukuserver] Read Only Mode

[GreenLunar]

I read in several instances that the bukuserver is not for production, yet. I know.
bukuserver Web UI is useful even when it is in read-only mode, especially with portable devices, such as PDAs etc.

Please add a read-only mode to bukuserver.

[GreenLunar]

FYI: I've opened a feature request at SearXNG and YaCy search engine projects.

[LeXofLeviafan]

…This one is pretty trivial, since flask-admin has direct support for disabling edit actions.

[jarun]

Closing as doing it from the above layer makes more sense.

[LeXofLeviafan]

doing it from the above layer

WDYM?

[jarun]

From your comment my understanding was it can be done using some configuration provided by the flask framework. If that's not the case and code change is required, please reopen and raise the PR.

[LeXofLeviafan]

Ah… The disabling itself is done via BaseModelView fields, on per-model basis.

[LeXofLeviafan]

(…As for reopening the issue, I don't seem to have the necessary access to do so – at least there doesn't seem to be such an action available in GitHub UI)

[rachmadaniHaryono]

just create pr and link to the issue, i will reopen it

[jarun]

Thanks @rachmadaniHaryono!

@LeXofLeviafan I have added you as a collaborator. Please discuss before merging any PR and avoid direct pushes.

[LeXofLeviafan]

You know you can just disable direct pushes to master, right? :-)

[jarun]

Sometimes I need that myself at wee hours for changes in the cli side.

[LeXofLeviafan]

In cases it's necessary I just re-enable direct pushes for a short time ;-)

[LeXofLeviafan]

…Might also be a good idea to set up mandatory approval before merging (though I seem to recall there being some bugs in GitHub related to this functionality).

[jarun]

Ignore the admin stuff... let's get back to what matters.