Skip to content

add screen_name to popup url? #23

@Serexx

Description

@Serexx

Hey - appreciate the work, saved me a lot of fiddling, just one request :-)

As per here:
https://developer.twitter.com/en/docs/basics/authentication/api-reference/authenticate
we would like pass a screen_name param to the popup URL to prefill the screen_name input.

I think, its just a matter of adding it to TwitterLogin.propTypes and appending it to the popup url like so

popup.location = https://api.twitter.com/oauth/authenticate?oauth_token=${data.oauth_token}&force_login=${this.props.forceLogin}&screen_name=${this.props.screenName||""}`;`

if the property is not supplied the || or just passes the empty string, with no visible effect.

I'm going to clone it and give it a try (any reason I shouldn't ?) - if it works as expected do you want a pull request?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions