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

6173 fix settings api #6176

Merged
merged 2 commits into from
Sep 17, 2019
Merged

6173 fix settings api #6176

merged 2 commits into from
Sep 17, 2019

Conversation

landreev
Copy link
Contributor

@landreev landreev commented Sep 17, 2019

New Contributors

Welcome! New contributors should at least glance at CONTRIBUTING.md, especially the section on pull requests where we encourage you to reach out to other developers before you start coding. Also, please note that we measure code coverage and prefer you write unit tests. Pull requests can still be reviewed without tests or completion of the checklist outlined below. Thanks!

Related Issues

Pull Request Checklist

@dataversebot
Copy link

Can one of the admins verify this patch?

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.006%) to 19.508% when pulling 83de8c0 on 6173-fix-settings-api into cf4b864 on develop.

@pdurbin
Copy link
Member

pdurbin commented Sep 17, 2019

Tested with 6173-fix-settings-api-83de8c0 spun up with this from dataverse-sample-data:

./ec2-create-instance.sh -g ec2config.yaml -a dataverse-6142-solr-schema-changes

@kcondon kcondon self-assigned this Sep 17, 2019
@kcondon kcondon merged commit 210499a into develop Sep 17, 2019
@kcondon kcondon deleted the 6173-fix-settings-api branch September 17, 2019 23:58
@djbrooke djbrooke added this to the 4.17 milestone Sep 18, 2019
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.

Settings API broken? publishing failing on phoenix server, API test suite down
6 participants