Updated to the GET /handlers Endpoint to Support Forms in the GUI #8911
+94
−0
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.
Description
At the moment, through our GUI, it is not possible to initialize all of our integrations through the forms section. The reason for this is because certain information that is required for the form is maintained as constants here in the front-end repository.
It would be more robust and extensible if this information can be delivered through our API endpoints and the objective of this PR is to accomplish that. This is done by introducing the following new attributes to our existing
GET /handlers
endpoint,most_popular
: determined by querying the top 10 data sources and AI engines as per our data warehouse in BigQuery.is_aws
: determined checking certain keywords against the title of the integration.is_new
: determined by checking the creation/modification date of the about.py file of each integration.Is this a good way to go about it? The additional computation will require a little more time for this endpoint to return a response. Should this maybe be separated out to a new endpoint?
Type of change
Verification Process
To ensure the changes are working as expected:
Checklist: