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

Docs: Poor documentation over Sandbox feature #535

Closed
haroldolivieri opened this issue Sep 3, 2019 · 5 comments
Closed

Docs: Poor documentation over Sandbox feature #535

haroldolivieri opened this issue Sep 3, 2019 · 5 comments

Comments

@haroldolivieri
Copy link

haroldolivieri commented Sep 3, 2019

First of all, great tool.
The potential here is amazing.

Secondly, it can be totally my lack of attention but it is not clear at all how to set up a SandboxFlipperPluginStrategy.

What is expected to be the keys and values in getKnownSandboxes()'s map and what does the method setSandbox()?

@kxfang
Copy link

kxfang commented Sep 3, 2019

Also it looks like the Sandbox plugin is disabled by GK. Is that intentional?

image

@jknoxville
Copy link
Contributor

From looking at the code, I believe you just need to implement a SandboxFlipperPluginStrategy, and pass it in, when constructing the plugin.

setSandbox should override the sandbox within your app. It will be called on your strategy when the user enters a string as their sandbox. What this does is technically up to the implementation, but it could for example, set the base url for outgoing requests from the app to staging.myapp.com for example. How this is done will depends on your app.

getKnownSandboxes should return a map of predefined options, keyed by name. For example, you could provide this as a map:

{
test: test.myapp.com,
staging: staging.myapp.com,
production: prod.myapp.com
}

to let the user select from a list without typing.

Feel free to update the docs:
https://github.com/facebook/flipper/blob/master/docs/setup/sandbox-plugin.md

jknoxville added a commit that referenced this issue Sep 20, 2019
Removes a gatekeeper that shouldn't be there.
Reported in #535
facebook-github-bot pushed a commit that referenced this issue Sep 23, 2019
Summary:
Removes a gatekeeper that shouldn't be there.
Reported in #535
Pull Request resolved: #554

Reviewed By: danielbuechele

Differential Revision: D17502725

Pulled By: jknoxville

fbshipit-source-id: 1865db3f5c8cd81b0ec747687d0a7312e5f2332b
@jbfitb
Copy link
Contributor

jbfitb commented Mar 2, 2020

I came here to ask what the Sandbox plugin does. Docs are pretty sparse. Happy to update the docs if you tell me what to say 😄

https://fbflipper.com/docs/setup/sandbox-plugin.html

@stale
Copy link

stale bot commented May 31, 2020

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community's attention? This issue may be closed if no further activity occurs. You may also label this issue as a "Discussion" or add it to the "Backlog" and I will leave it open. Thank you for your contributions.

@stale stale bot added the Stale label May 31, 2020
@stale
Copy link

stale bot commented Jun 14, 2020

Closing this issue after a prolonged period of inactivity. If this issue is still present in the latest release, please feel free to create a new issue with up-to-date information.

@stale stale bot closed this as completed Jun 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants