Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@pedronauck
Copy link
Contributor

@pedronauck pedronauck commented Mar 13, 2024

This PR includes social login for claiming faucet tokens using clerk as a provider.
Also includes code splitting.

@pedronauck pedronauck self-assigned this Mar 13, 2024
@luizstacio luizstacio requested a review from xgreenx March 14, 2024 22:58
# Conflicts:
#	src/lib.rs
#	src/models.rs
@MujkicA MujkicA self-assigned this Mar 25, 2024
fuel_faucet_max_dispenses_per_minute=20
fuel_faucet_min_gas_price=0
fuel_faucet_dispense_amount=10000
fuel_faucet_clerk_pub_key="6Ld3cEwfAAAAAMd4QTs7aO85LyKGdgj0bFsdBfre"
Copy link
Contributor

Choose a reason for hiding this comment

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

What values should be used here?

@MujkicA MujkicA requested a review from xgreenx March 28, 2024 14:41
@MujkicA
Copy link
Contributor

MujkicA commented Mar 28, 2024

I've fixed the tests and made some light adjustments.
Also, I've changed the validate endpoint to store the user_id in the session so that we can save one network call to Clerk.

Shoutout to @nedsalk for helping me pinpoint some errors related to frontend/backend cookie handling.

Copy link
Contributor

@xgreenx xgreenx left a comment

Choose a reason for hiding this comment

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

Look good to me! I only have several questions for @luizstacio I will ask them in Slack

}

#[derive(Debug, Clone)]
struct MockAuthHandler {
Copy link
Contributor

Choose a reason for hiding this comment

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

Very nice way to bypass registration in the tests!=)

@pedronauck pedronauck merged commit 7175ccb into master Apr 8, 2024
@pedronauck pedronauck deleted the pn/feat/claim-auth branch April 8, 2024 16:49
xgreenx added a commit that referenced this pull request Apr 9, 2024
xgreenx added a commit that referenced this pull request Apr 22, 2024
Revert "feat: add claim using auth (#61)"

This reverts commit 7175ccb.
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.

6 participants