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

Stellar | GIVback eligiblity #4582

Closed
maryjaf opened this issue Aug 18, 2024 · 16 comments
Closed

Stellar | GIVback eligiblity #4582

maryjaf opened this issue Aug 18, 2024 · 16 comments
Assignees
Labels

Comments

@maryjaf
Copy link
Collaborator

maryjaf commented Aug 18, 2024

"Is givback eligible" is set for this token https://impact-graph.serve.giveth.io/admin/resources/Token/records/306/show
but I haven't seen the donation with stellar in response of this API

curl -X 'GET'
'https://givback.develop.giveth.io/eligible-donations?chain=all-other-chains&startDate=2024%2F08%2F18-00%3A00%3A00&endDate=2024%2F08%2F19-23%3A59%3A59&download=no'
-H 'accept: application/json'

cc: @MoeNick @Meriem-B

@maryjaf maryjaf added this to All-Devs Aug 18, 2024
@github-project-automation github-project-automation bot moved this to New Issues in All-Devs Aug 18, 2024
@MoeNick
Copy link
Member

MoeNick commented Aug 19, 2024

Also, the icon for Stellar should be demonstrated with a tiny GIVback hand icon.

@maryjaf
Copy link
Collaborator Author

maryjaf commented Aug 20, 2024

This issue hasn't been assigned yet

@mohammadranjbarz mohammadranjbarz self-assigned this Aug 20, 2024
@mohammadranjbarz
Copy link
Contributor

@maryjaf @MoeNick Now we have some restrictions that send givbacks for EVM donations, should we change it?
Because if we change it to support non-EVM donations we need to find a solution to send givbacks for them as the script and the format that we use for sending givbacks is only available on Gnosis and OP chain.
Correct me if I'm wrong @laurenluz @WhyldWanderer

@GriffGreen
Copy link
Member

GriffGreen commented Aug 20, 2024

We would send GIVbacks on OP, not on Stellar, like we do for donations on Arbitrum, the main thing is that it will only work if they are logged in with a Ethereum profile (not a solana profile)

@mohammadranjbarz
Copy link
Contributor

We would send GIVbacks on OP, not on Stellar, like we do for donations on Arbitrum, the main thing is that it will only work if they are logged in with a Ethereum profile (not a solana profile)

Thanks @GriffGreen , will work on that

@mohammadranjbarz mohammadranjbarz moved this from New Issues to In Progress in All-Devs Aug 26, 2024
mohammadranjbarz added a commit to Giveth/givback-calculation that referenced this issue Aug 28, 2024
@mohammadranjbarz mohammadranjbarz moved this from In Progress to Code Review/PR in All-Devs Aug 28, 2024
@mohammadranjbarz mohammadranjbarz moved this from Code Review/PR to QA in All-Devs Aug 28, 2024
@mohammadranjbarz
Copy link
Contributor

FYI @maryjaf @MoeNick it's deployed on Staging

@maryjaf
Copy link
Collaborator Author

maryjaf commented Aug 29, 2024

but I haven't seen the donation with stellar in response of this API

curl -X 'GET' 'https://givback.develop.giveth.io/eligible-donations?chain=all-other-chains&startDate=2024%2F08%2F18-00%3A00%3A00&endDate=2024%2F08%2F19-23%3A59%3A59&download=no' -H 'accept: application/json'

So I should see the stellar donation in response of this api now? @mohammadranjbarz

@divine-comedian
Copy link
Contributor

Hey guys! I was testing the flow today and wanted to add a comment related to this issue:

For me it wasn't obvious that if I'm not signed in with my EVM profile I won't get GIVbacks! It would be nice to remind me if I'm not signed in that I should do it, otherwise I'll miss out on GIVbacks!

Image

@divine-comedian
Copy link
Contributor

divine-comedian commented Aug 29, 2024

Also! If I'm not signed in and make a successful donation I shouldn't see the GIVbacks component on the donation success page, since I won't actually get GIVbacks

Image

@maryjaf
Copy link
Collaborator Author

maryjaf commented Sep 1, 2024

but I haven't seen the donation with stellar in response of this API
curl -X 'GET' 'https://givback.develop.giveth.io/eligible-donations?chain=all-other-chains&startDate=2024%2F08%2F18-00%3A00%3A00&endDate=2024%2F08%2F19-23%3A59%3A59&download=no' -H 'accept: application/json'

So I should see the stellar donation in response of this api now? @mohammadranjbarz

{
"amount": 0.1,
"currency": "XLM",
"createdAt": "2024-09-01-07:49:00",
"valueUsd": 0.0092289,
"bottomRankInRound": null,
"givbacksRound": null,
"projectRank": null,
"givbackFactor": 0.75,
"valueUsdAfterGivbackFactor": 0,
"giverAddress": "0xa047b554b435674eb28fd4553201cb1a800be53b",
"txHash": "0383be71d2d726233e09477973d44ac8338ea7ce9df10f4faca405d202d3306c",
"network": "stellar",
"source": "giveth.io",
"giverName": "testaccoun2 mozila",
"giverEmail": null,
"projectLink": "https://giveth.io/project/test-stellar",
"isReferrerGivbackEligible": false,
"referrerWallet": null
},

I donated to test stellar project, and above record is shown in response of givback api.
Is the value of these below fields correct? @mohammadranjbarz

"bottomRankInRound": null,
"givbacksRound": null,
"valueUsdAfterGivbackFactor": 0,

@MoeNick
Copy link
Member

MoeNick commented Sep 1, 2024

Thanks, @divine-comedian for raising those issues, those will be addressed on the issues above so as not to distract the GIVback API QA.

@maryjaf
Copy link
Collaborator Author

maryjaf commented Sep 2, 2024

I donated to test stellar project, and above record is shown in response of givback api. Is the value of these below fields correct? @mohammadranjbarz

"bottomRankInRound": null, "givbacksRound": null, "valueUsdAfterGivbackFactor": 0,

@mohammadranjbarz @Meriem-B

@mohammadranjbarz
Copy link
Contributor

0383be71d2d726233e09477973d44ac8338ea7ce9df10f4faca405d202d3306c

I checked the stellar donations on staging, the rank info is null for all of them as you see in this screenshot

Screenshot 1403-06-13 at 11 37 17 in the morning

@Meriem-B Can you investigate it? I spent about 1 hour but couldn't find the reason, if you could not find it as well, please write some test cases for this scenario, reproduce it in test cases and then fix it ( Ask me if you need help)

@maryjaf maryjaf moved this from QA to In Progress in All-Devs Sep 3, 2024
@mohammadranjbarz mohammadranjbarz moved this from In Progress to QA in All-Devs Sep 3, 2024
@mohammadranjbarz
Copy link
Contributor

@maryjaf can you test it again?
It should work for new donations that you will create from now on

@maryjaf
Copy link
Collaborator Author

maryjaf commented Sep 3, 2024

Now this record is shown in response of givback eligibility
It seems the problem has been fixed @mohammadranjbarz

"givbacksRound": 49443 is this filed filled correctly?

{
"amount": 0.1,
"currency": "XLM",
"createdAt": "2024-09-03-09:53:00",
"valueUsd": 0.0093546,
"bottomRankInRound": 33,
"givbacksRound": 49443,
"projectRank": 33,
"givbackFactor": 0.5,
"valueUsdAfterGivbackFactor": 0.0046773,
"giverAddress": "0x7f37e3008207c27360b20abcfb5fdcc8e37596b8",
"txHash": "d02a1bf00da3ffcc344d4700ff2dfadfe3d28bd981a8765565a8fcf27babd305",
"network": "stellar",
"source": "giveth.io",
"giverName": "maryamaa mehr",
"giverEmail": null,
"projectLink": "https://giveth.io/project/test-stellar",
"isReferrerGivbackEligible": false,
"referrerWallet": null
},

@mohammadranjbarz
Copy link
Contributor

mohammadranjbarz commented Sep 3, 2024

w

Now this record is shown in response of givback eligibility It seems the problem has been fixed @mohammadranjbarz

"givbacksRound": 49443 is this filed filled correctly?

{ "amount": 0.1, "currency": "XLM", "createdAt": "2024-09-03-09:53:00", "valueUsd": 0.0093546, "bottomRankInRound": 33, "givbacksRound": 49443, "projectRank": 33, "givbackFactor": 0.5, "valueUsdAfterGivbackFactor": 0.0046773, "giverAddress": "0x7f37e3008207c27360b20abcfb5fdcc8e37596b8", "txHash": "d02a1bf00da3ffcc344d4700ff2dfadfe3d28bd981a8765565a8fcf27babd305", "network": "stellar", "source": "giveth.io", "giverName": "maryamaa mehr", "giverEmail": null, "projectLink": "https://giveth.io/project/test-stellar", "isReferrerGivbackEligible": false, "referrerWallet": null },

Thanks @maryjaf yes it's ok

@maryjaf maryjaf moved this from QA to Done in All-Devs Sep 3, 2024
@divine-comedian divine-comedian closed this as completed by moving to Merged to Production in All-Devs Oct 3, 2024
@github-project-automation github-project-automation bot moved this from Merged to Production to Done in All-Devs Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

No branches or pull requests

5 participants