Skip to content
This repository has been archived by the owner on Sep 21, 2023. It is now read-only.

Client-side UI #14

Closed
wants to merge 14 commits into from
Closed

Client-side UI #14

wants to merge 14 commits into from

Conversation

alecmerdler
Copy link

@alecmerdler alecmerdler commented Apr 3, 2017

Adds a web UI using the generate command, which allows the user to enter a Stripe private key, choose permissions, and generate signed credentials. This is achieved purely client-side with no calls to the server using Crypto-JS and AngularJS.

Addresses #4

screenshot_20170404_215759

@alecmerdler alecmerdler requested a review from jakedt April 3, 2017 10:28
Copy link
Contributor

@jakedt jakedt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we:

  • Move this to a new subcommand, maybe generate
  • Have the credentials live update when the permissions are changes
  • Move the clear all button to the top
  • When you click ResourceAll, have the checkboxes below it show up checked and disabled, but not actually be checked in terms of the credentials. ResourceAll is a superset but only in terms of the application of auth.

@jakedt
Copy link
Contributor

jakedt commented Apr 3, 2017

  • Also need to update the README with instructions about how to use the new server to generate credentials.

@alecmerdler
Copy link
Author

Okay, I have made the changes you requested. I'm still new to Go, so let me know if I made any mistakes.

@jakedt
Copy link
Contributor

jakedt commented Apr 4, 2017

Almost there! For some reason starting at about the 5th row down (in both columns) changing the state of the checkbox doesn't update the generated key.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants