Skip to content
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

Merged
merged 14 commits into from
Feb 8, 2021

Conversation

MaggieKimani1
Copy link
Contributor

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

@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2021

Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-829.centralus.azurestaticapps.net

1 similar comment
@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2021

Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-829.centralus.azurestaticapps.net

src/index.tsx Outdated Show resolved Hide resolved
@github-actions
Copy link
Contributor

github-actions bot commented Feb 8, 2021

Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-829.centralus.azurestaticapps.net

1 similar comment
@github-actions
Copy link
Contributor

github-actions bot commented Feb 8, 2021

Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-829.centralus.azurestaticapps.net

@github-actions
Copy link
Contributor

github-actions bot commented Feb 8, 2021

Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-829.centralus.azurestaticapps.net

@github-actions
Copy link
Contributor

github-actions bot commented Feb 8, 2021

Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-829.centralus.azurestaticapps.net

MaggieKimani1 added a commit that referenced this pull request Feb 8, 2021
* 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.
MaggieKimani1 added a commit that referenced this pull request Feb 8, 2021
MaggieKimani1 added a commit that referenced this pull request Feb 9, 2021
* 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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants