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

Allow submitting more logs types #400

Closed
andrixnet opened this issue Mar 31, 2016 · 9 comments
Closed

Allow submitting more logs types #400

andrixnet opened this issue Mar 31, 2016 · 9 comments

Comments

@andrixnet
Copy link
Contributor

Ref: cgeo/cgeo#5539 (comment)
Ref: opencaching/opencaching-pl#412
Ref: #398

According to c:geo developers, some log types cannot be submitted via c:geo to OPENCACHING-PL websites because OKAPI does not support them.

This is a feature request that OKAPI implement log retrieval and log submission for all log types supported by OPENCACHING-PL (at this time, with changes and addition in the future, as they come).

@following5 following5 changed the title Implement all log types supported by OPENCACHING-PL Implement OCPL "Maintenance performed" logs Mar 31, 2016
@following5
Copy link
Contributor

There is already the issue #260 for implementing log types which may change the status of the cache.

This issue (#400) probably is about the OCPL log type "Maintenance performed", which does not change the geocache status and therefor may be handled separately.

@andrixnet
Copy link
Contributor Author

Then this is also about log type "maintenance required", which also does not change cache status.

@following5
Copy link
Contributor

Then this is also about log type "maintenance required", which also does not change cache status.

That's already available in OKAPI, and I have already opened an issue in the c:geo project so that it may be implemented there. You have referenced it above.

@andrixnet
Copy link
Contributor Author

        if (
            $needs_maintenance2 == 'false'
            && Settings::get('OC_BRANCH') == 'oc.pl'
        ) {
            # If not supported, just ignore it.

            self::$success_message .= " ".sprintf(_(
                "However, your \"does not need maintenance\" flag was ignored, because ".
                "%s does not yet support this feature."
            ), Okapi::get_normalized_site_name());
            $needs_maintenance2 = 'null';
        } 

Is this correct?
Shouldn't the string read:

However, your "needs maintenance" flag was ignored, because %s does not yet support this feature.

or

However, your "maintenance performed" flag was ignored, because %s does not yet support this feature.

From what I read in submit.php and this issue, this would seem the logical text.

@following5
Copy link
Contributor

Did you also read the docs on 'needs_maintenance2'?

@wrygiel
Copy link
Member

wrygiel commented Apr 1, 2016

Broken link ;) This one should work: http://www.opencaching.de/okapi/services/logs/submit.html

@andrixnet - it seems correct to me.

@andrixnet
Copy link
Contributor Author

Then my understanding was incorrect. Thank you for the context clarification.

@andrixnet
Copy link
Contributor Author

RO translations: #404

@wrygiel wrygiel changed the title Implement OCPL "Maintenance performed" logs Allow submitting more logs types Oct 6, 2016
@following5
Copy link
Contributor

following5 commented Oct 31, 2018

Split into two issues, #260 for status-changing logs and #548 for OCPL "maintenance performed" logs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants