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

Add git cion score to donation flow #142

Merged
merged 18 commits into from
Nov 26, 2024
Merged

Conversation

ae2079
Copy link
Collaborator

@ae2079 ae2079 commented Nov 24, 2024

No description provided.

Copy link
Collaborator

@MohammadPCh MohammadPCh left a comment

Choose a reason for hiding this comment

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

@ae2079 Thanks for the PR! It looks great to me, but since Carlos is the HERO of this part, I’d recommend having him review it as well to ensure everything is spot on. 😊

@MohammadPCh
Copy link
Collaborator

Also, @aminlatifi is the Hero of this part

Copy link

@aminlatifi aminlatifi left a comment

Choose a reason for hiding this comment

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

Please update the name of variables to be better self explainatory

config/example.env Outdated Show resolved Hide resolved
config/example.env Outdated Show resolved Hide resolved
config/example.env Outdated Show resolved Hide resolved
src/entities/user.ts Outdated Show resolved Hide resolved
src/utils/qacc.ts Show resolved Hide resolved
src/services/qAccService.ts Outdated Show resolved Hide resolved
src/services/qAccService.ts Outdated Show resolved Hide resolved
src/services/qAccService.ts Outdated Show resolved Hide resolved
src/services/qAccService.ts Outdated Show resolved Hide resolved
src/resolvers/donationResolver.ts Outdated Show resolved Hide resolved
src/resolvers/donationResolver.ts Outdated Show resolved Hide resolved
src/resolvers/donationResolver.ts Outdated Show resolved Hide resolved
change env variables names

Co-authored-by: Amin Latifi <[email protected]>
@aminlatifi
Copy link

src/entities/user.ts Outdated Show resolved Hide resolved
src/entities/user.ts Outdated Show resolved Hide resolved
src/resolvers/qAccResolver.ts Outdated Show resolved Hide resolved
src/services/qAccService.ts Outdated Show resolved Hide resolved
src/services/qAccService.ts Outdated Show resolved Hide resolved
src/services/userService.ts Outdated Show resolved Hide resolved
Copy link

@aminlatifi aminlatifi left a comment

Choose a reason for hiding this comment

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

Thanks @ae2079
Great job

GITCOIN_PASSPORT_MIN_VALID_ANALYSIS_SCORE=
# 1 day
GITCOIN_PASSPORT_EXPIRATION_PERIOD_MS=86400000
MAX_CONTRIBUTION_WITH_GITCOIN_PASSPORT_ONLY_IN_USD=

Choose a reason for hiding this comment

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

Suggested change
MAX_CONTRIBUTION_WITH_GITCOIN_PASSPORT_ONLY_IN_USD=
MAX_CONTRIBUTION_WITH_GITCOIN_PASSPORT_ONLY_USD=

@ae2079 ae2079 merged commit 14104b4 into staging Nov 26, 2024
8 checks passed
@ae2079 ae2079 deleted the addGitCionScoreToDonationFlow branch November 26, 2024 14:15
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.

3 participants