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

Put validator type into the context #9

Merged
merged 3 commits into from
Mar 1, 2019
Merged

Conversation

Bjwebb
Copy link
Member

@Bjwebb Bjwebb commented Feb 25, 2019

This is required for
OpenDataServices/cove#1117

@Bjwebb

This comment has been minimized.

@Bjwebb Bjwebb force-pushed the cove-1117-group-validation branch from 80e2f53 to e53cc67 Compare February 28, 2019 07:12
@BibianaC
Copy link
Member

@Bjwebb should this be merged in master or in the temp master branch?

@Bjwebb
Copy link
Member Author

Bjwebb commented Feb 28, 2019

We don't have a temp master branch for this repo (lib-cove), so I think its simplest to merge into master. This change shouldn't have any noticeable effect without the corresponding cove change.

@Bjwebb Bjwebb merged commit 2fee8ee into master Mar 1, 2019
@Bjwebb Bjwebb deleted the cove-1117-group-validation branch March 1, 2019 08:22
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.

2 participants