-
Notifications
You must be signed in to change notification settings - Fork 93
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
Feature: Enable loading of samples and Permissions descriptions files from Github #829
Conversation
…he query string and append them to devxapi object
Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-829.centralus.azurestaticapps.net |
1 similar comment
Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-829.centralus.azurestaticapps.net |
…ature/samples-testing
Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-829.centralus.azurestaticapps.net |
1 similar comment
Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-829.centralus.azurestaticapps.net |
Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-829.centralus.azurestaticapps.net |
Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-829.centralus.azurestaticapps.net |
…esting" This reverts commit 220bacb.
* Check for devxApiUrl override, fetch org and branchName params from the query string and append them to devxapi object * Change response type to object * Create an initialState object and pass it to state * Add check for devxapi params and append them to samplesUrl * Check for devxapi params and append them to the permissionsUrl * Create an IDevxAPI interface * Import IDevxAPI from types and reference it * Import and reference IAction from the types folder * Rename the devx-api types file * Use ternary operator to simplify conditional statement * Revert "Merge pull request #829 from microsoftgraph/feature/samples-testing" This reverts commit 220bacb.
…amples-testing"" This reverts commit 2839dce.
* Check for devxApiUrl override, fetch org and branchName params from the query string and append them to devxapi object * Change response type to object * Create an initialState object and pass it to state * Add check for devxapi params and append them to samplesUrl * Check for devxapi params and append them to the permissionsUrl * Create an IDevxAPI interface * Import IDevxAPI from types and reference it * Import and reference IAction from the types folder * Rename the devx-api types file * Use ternary operator to simplify conditional statement * Revert "Merge pull request #829 from microsoftgraph/feature/samples-testing" This reverts commit 220bacb. * Revert "Revert "Merge pull request #829 from microsoftgraph/feature/samples-testing"" This reverts commit 2839dce.
Overview
This PR proposes adding of testing functionality for samples and permissions descriptions files in the DevX Content repo
Testing Instructions
Run Graph Explorer locally on your machine
Clone https://github.com/microsoftgraph/microsoft-graph-explorer-api and Run DevX API locally
Override fetching from blob storage by calling and supplying org and branchName params to the devx-api query string
http://localhost:3000/?devx-api=https://localhost:5001&org=mg&branchName=dev
The call should return a set of samples and permissions from the DevX Content repo