Skip to content
This repository was archived by the owner on Aug 30, 2023. It is now read-only.

Add search api helper #37

Merged
merged 6 commits into from
Sep 9, 2021
Merged

Add search api helper #37

merged 6 commits into from
Sep 9, 2021

Conversation

ianwow
Copy link
Member

@ianwow ianwow commented Sep 8, 2021

Issue #, if available:

Description of changes:

Add a modal in the GUI to show users how to use the search api via the command line.

It will look like this:

Screen Shot 2021-09-09 at 8 25 22 AM

The modal will provide a sample awscurl command which users can copy and paste into their command line to run, like this:

awscurl -X GET --profile default --service es --region us-west-2 'https://search-prb2e34-esdoma-1meb6f6ir8rnc-jx6sarrj4ohpgtq4cvkoixvzle.us-west-2.es.amazonaws.com/_search?q=AssetId%3A2c444585-1a85-4913-b37f-45a24c6a67e3%20Operator%3Atechnical_cue_detection'

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@@ -77,8 +77,9 @@ export default {
},
markers: []
});
this.player.autoplay('muted');
this.player.currentTime(0);
// Set player time to 2 second mark in order to avoid a black poster image for
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm including this small change in this PR even though it's unrelated to the topic of the PR. This is such a small change that I don't think it needs a separate PR.

Copy link

@aburkleaux-amazon aburkleaux-amazon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding this! It will be very useful for developers using CAS as a sample application. I'm entering an issue to merge it to Content Localization as well.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants