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

Add suggested edits endpoints, take 2 #1159

Closed

Conversation

mdholloway
Copy link
Contributor

Adds four new recommendation endpoints:

  • /data/recommendation/caption/addition/{target}
  • /data/recommendation/caption/translation/from/{source}/to/{target}
  • /data/recommendation/description/addition/{target}
  • /data/recommendation/description/translation/from/{source}/to/{target}

The caption suggestion endpoints are supported only for Commons domains,
and the description suggestion endpoints are supported only for Wikidata
domains. (This is enforced in the service configuration, and is not reflected here.)

Support for commons.wikimedia.org as a default project is added to
config.example.wikimedia.yaml for development and testing.

Support for recommendations endpoints is added to the wikidata.org
project in the example config.

Phab: https://phabricator.wikimedia.org/T224754

(Note: This is a continuation of #1153 after rewriting some history in a way Git doesn't necessarily appreciate.)

mdholloway and others added 6 commits July 2, 2019 14:24
Adds four new recommendation endpoints:

- /data/recommendation/caption/addition/{target}
- /data/recommendation/caption/translation/from/{source}/to/{target}
- /data/recommendation/description/addition/{target}
- /data/recommendation/description/translation/from/{source}/to/{target}

The caption suggestion endpoints are supported only for Commons domains,
and the description suggestion endpoints are supported only for Wikidata
domains.

Support for commons.wikimedia.org as a default project is added to
config.example.wikimedia.yaml for development and testing.

Support for recommendations endpoints is added to the wikidata.org
project in the example config.

Phab: https://phabricator.wikimedia.org/T224754
1- Rebase.
2- Split recommend configurations and add into projects wikidata.wmf and
commons.wmf.
3- Create commons.wmf per copy of default.wmf and add recommend-caption
entries.
1- Remove commons from projects and add recommend-caption direct in the
config file
2- Add wikidata and commons to the monitoring test script as allowed
domains
These endpoints will be monitored at the service level and do not need
additional monitoring here.
@mdholloway mdholloway changed the title Suggested edits updated Add suggested edits endpoints, take 2 Jul 2, 2019
- path: projects/v1/default.wmf.yaml
options: *default_options
- path: v1/recommend-caption.yaml
options: '{{options.recommendation}}'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't work because options is 'undefined' when declared like this. The right thing to do is done in the other PR https://github.com/wikimedia/restbase/pull/1153/files#diff-b91bbd009bed6f8dfe87647993d47521R119

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All right, I'll try picking that one back up

@mdholloway mdholloway closed this Jul 3, 2019
@mdholloway mdholloway reopened this Jul 3, 2019
@mdholloway mdholloway closed this Jul 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants