-
Notifications
You must be signed in to change notification settings - Fork 0
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
[BI-1609] Add configuration to enable/disable PUT/POST /variables endpoints in Breedbase #95
Comments
This is a feature that I was trying to add to the BrAPI spec to all endpoints long ago. @BrapiCoordinatorSelby hope this can go to the serverInfo this time. |
@MFlores2021 I think we have a misunderstanding here? |
It is related I believe. I meant I was enabling and disabling PUT / POST endpoints in the server side. And reflecting that availability by permissions in the serverInfo endpoint. I think availability shouldn't be binary it should be granted by user and it should be shown in serverInfo. |
Thanks for the clarification, I see what you mean now! And yes it is related, but I would argue it is not dependent on BrAPI. This functionality could be added to Breedbase before there is a place in the BrAPI spec to represent it, correct? I just don't want the BrAPI spec to be a blocker for new functionality. I haven't even started looking at the next version of the spec yet, so it will be a long time more before permissions can be officially represented in |
@BrapiCoordinatorSelby BrAPI spec is not a blocker for sure!. I just made the point to keep in mind, no worries. |
Update BreedBase to allow for the POST /variables and PUT /variables endpoints to be enabled or disabled based on a configuration property.
Jira story: https://breedinginsight.atlassian.net/browse/BI-1609
Techspecs
The text was updated successfully, but these errors were encountered: