added indexname to the toggle request interface #192
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently, there is a bug in the app where if you try to toggle off/on a specialization, the fields Document Count, Strictness, Past messages included and MaxTokenResponse revert back to default. The reason this is happening is because when we toggle the specialization, qSpecializationmutate only has the isActive field set and everything else is null. Because of that when we create the response, since the index name is null, the aforementioned fields are set to null because these fields should not be set without an index. This behavior is somewhat expected because we have a 'None' field in index name because of general being a specialization. So I am passing the index name in the Toggle request interface to distinguish between index being updated and just a specialization toggle.
Contribution Checklist