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

initial fully customized sign in and registration example #92

Merged
merged 12 commits into from
Jul 10, 2020

Conversation

merlinoa
Copy link
Collaborator

@merlinoa merlinoa commented Jul 8, 2020

@phoward can you please take a look at this?

It adds a more formal structure to creating fully customized sign in and registration pages with polished. I think it still needs a little work though. Can you follow the new vignette and try and set up a fully customize sign in + registration page and let me know what you think.

@merlinoa merlinoa requested a review from phoward38 July 8, 2020 20:32
@phoward38 phoward38 self-assigned this Jul 8, 2020
@phoward38 phoward38 added the p1 label Jul 8, 2020
@phoward38
Copy link
Contributor

Fixed working example w/ @merlinoa, then:

  • Updated vignette
  • IMPORTANT: We're requiring users to pass sign_in as the ID for the custom UI module
    • Added note to vignette & reflected in example

Other:

  • Added action link for switching between sign in & register
  • Added default label to email_input (keep consistent w/ password_input)

R/email_input.R Outdated
email_input <- function (inputId, label, value = "", width = NULL, placeholder = NULL) {
email_input <- function (
inputId,
label = tagList(icon("envelope"), "Email"),
Copy link
Contributor

Choose a reason for hiding this comment

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

Added default label like password_input

- "register_password_verify"
- "submit_register"

And when passing your custom module UI as the `sign_in_page_ui` argument in the `secure_ui` function, you **must** use `sign_in` as the UI module's ID (as shown in the example below).
Copy link
Contributor

Choose a reason for hiding this comment

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

Added note about required sign_in ID for custom UI module

@phoward38 phoward38 added andy_review and removed p1 labels Jul 9, 2020
@phoward38 phoward38 assigned merlinoa and phoward38 and unassigned phoward38 Jul 9, 2020
@merlinoa merlinoa merged commit cc23be3 into master Jul 10, 2020
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