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.

Sign up for GitHub

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

Jump to bottom

CLI demo #52

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

CLI demo #52

wants to merge 6 commits into from

Conversation

bkeepers
Copy link
Collaborator

@bkeepers bkeepers commented Feb 8, 2024 •
edited
Loading

This is an experiment to make a web-based demo of the $ flipper CLI. It's not perfect, but works surprisingly well.

2024-02-08 14 38 43

@jnunemaker
Copy link
Contributor

Looks neat!

How does it work? Like does it somehow point to the same session and all that?

@bkeepers
Copy link
Collaborator Author

How does it work? Like does it somehow point to the same session and all that?

The middleware from #43 sets Flipper.instance to the session-based adapter, so any calls to Flipper.* act on that session adapter. So this just takes the user input and passes it directly to Flipper::CLI.run(args), which then calls Flipper.enable, Flipper.disable, etc.

@bkeepers bkeepers marked this pull request as ready for review February 12, 2024 17:43
@bkeepers
Copy link
Collaborator Author

I deployed this if you want to play with it:
https://playground.flippercloud.io/cli

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@garrettdimon garrettdimon Awaiting requested review from garrettdimon

@jnunemaker jnunemaker Awaiting requested review from jnunemaker

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bkeepers @jnunemaker