I added an instance variable to make this work in multi-tenancy #7
Conversation
|
We should remember to update the documentation once this goes in so that users are aware of the Additionally, do we want to default |
Add default as rapidapi if the parameter is not provided
|
@s-giovanni just added the change that should prevent existing integrations from breaking. Let me do another deeper dive before we merge though. |
LMS007
left a comment
LMS007
left a comment
There was a problem hiding this comment.
Thanks for taking a look at the working through this.
I also need to update the readme and we'll want to add a bit more functionality to include the POST vs GET options with a body payload for trigger overrides
I can take over from here!
| const LOCATION = core.getInput('location'); | ||
| console.log(`Executing In Location: ${LOCATION}`); | ||
|
|
||
| const INSTANCE = core.getInput('instance') || "rapidapi"; |
There was a problem hiding this comment.
TODO: define input in action.yml, possibly with a default "rapidapi.com"
| core.group('Execute Test', async () => { | ||
| // 1. Trigger Test | ||
| const envString = ENVIRONMENT ? `&enviroment=${ENVIRONMENT}` : ''; | ||
| const instance = INSTANCE; |
There was a problem hiding this comment.
I think if we make this the full domain w/ .com then we can just call this DOMAIN
No description provided.