Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Proposal for services/logs/edit and services/logs/delete #551

Merged
merged 4 commits into from
Nov 8, 2018

Conversation

following5
Copy link
Contributor

This adds a method for editing logs, as proposed in #421. Log type, comment and date can be changed.

Not implemented in this first version:

  • Changing from and to status log types like "Available". Users can submit a new log instead for correcting the geocache's status.
  • Changing rating or recommendation. This is ridiculously complex; I would rather wait if someone needs it. E.g. c:geo so far does not support ratings and recommendations at all.
  • Changing needs_maintenance. This is tricky, and we may want to solve Implement 'maintenance performed' logging for OCPL #548 first. Users may submit new a new log instead to indicate another maintenance state.

@following5 following5 changed the title Proposal for new services/logs/edit method Proposal for services/logs/edit and services/logs/delete Nov 6, 2018
@following5
Copy link
Contributor Author

Added proposal for services/logs/delete. So far it only allows deleting own logs. For delete-by-cacheowner, we need emails (#408).

@following5 following5 merged commit 51d7c9a into opencaching:master Nov 8, 2018
@following5 following5 deleted the editlog branch November 21, 2018 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant