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.

Sign up for GitHub

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

Jump to bottom

Fix harvest validation for accessLevel values #2263

Merged
merged 1 commit into from
Dec 1, 2017
Merged

Fix harvest validation for accessLevel values #2263

merged 1 commit into from
Dec 1, 2017

Conversation

janette
Copy link
Member

@janette janette commented Nov 30, 2017 •
edited by dafeder
Loading

When harvesting non-public or restricted public data the harvest error log will display validation errors for the accessLevel field.

errors___data_va_gov

QA Steps

  • Create a harvest source with "accessLevel": "non-public", and "accessLevel": "restricted public" values. Confirm there are no errors listed for accessLevel.

Reminders

  • There is test for the issue.
  • CHANGELOG updated.
  • Coding standards checked.
  • Review docs.getdkan.com (or in /docs) to see if it still covers the scope of the PR and update if needed.

@dafeder dafeder merged commit cf580b3 into 7.x-1.x Dec 1, 2017
@dafeder dafeder deleted the harvest-accessLevel-error branch December 1, 2017 17:05
dafeder pushed a commit that referenced this pull request Apr 24, 2020
@janette @dafeder
Fix harvest validation for accessLevel values (#2263)
7056e8d
dafeder pushed a commit that referenced this pull request Apr 24, 2020
@janette @dafeder
Fix harvest validation for accessLevel values (#2263)
49d9de7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@dafeder dafeder

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@janette @dafeder @kimwdavidson