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

Store creds frontend #213

Merged
merged 13 commits into from
May 12, 2020
Merged

Store creds frontend #213

merged 13 commits into from
May 12, 2020

Conversation

pregnantboy
Copy link
Contributor

Problem

Closes [insert issue #192]

Features:

UI features:

  • Accounts page
    • API generation
    • Storing new creds
  • Dropdown select on SMS credentials page
    • toggle between dropdown and manual input

Improvements:

  • Refactor
    • Separate twilio credentials input
    • Refactor SendModal to a common ConfirmModal

AFTER:
Screenshot 2020-05-11 at 3 39 21 PM
Screenshot 2020-05-11 at 3 41 03 PM
Screenshot 2020-05-11 at 3 39 34 PM
Screenshot 2020-05-11 at 3 40 21 PM
Screenshot 2020-05-11 at 3 40 27 PM
Screenshot 2020-05-11 at 3 40 44 PM
Screenshot 2020-05-11 at 3 40 51 PM
Screenshot 2020-05-11 at 3 55 42 PM

@pregnantboy pregnantboy force-pushed the store-creds-frontend branch from cb08d21 to cf17e8c Compare May 11, 2020 16:20
@pregnantboy pregnantboy force-pushed the store-creds-frontend branch 3 times, most recently from 8aa88fa to cf92870 Compare May 12, 2020 05:19
@pregnantboy pregnantboy force-pushed the store-creds-frontend branch from cf92870 to 66783e7 Compare May 12, 2020 06:10
@pregnantboy pregnantboy merged commit c3ee9a2 into develop May 12, 2020
@pregnantboy pregnantboy deleted the store-creds-frontend branch May 12, 2020 15:42
@pregnantboy pregnantboy mentioned this pull request May 19, 2020
Closed
@snyk-bot snyk-bot mentioned this pull request Mar 22, 2021
Merged
@snyk-bot snyk-bot mentioned this pull request Apr 28, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@pregnantboy