Skip to content
This repository was archived by the owner on Jun 24, 2022. It is now read-only.

[Affiliate] Referral data from prod server was added to referral address in dev server #1871

Closed
elena-zh opened this issue Nov 17, 2021 · 1 comment · Fixed by #1900
Closed
Assignees
Labels
app:CowSwap CowSwap app Bug Something isn't working Medium Severity indicator. It causes some undesirable behavior, but the system is still functional Protofire Handled by Protofire development team

Comments

@elena-zh
Copy link

Reported and discussed in #1869 (comment)

  1. I shared a referral link https://cowswap.staging.gnosisdev.com/#/?referral=0xFF714b8b0e2700303eC912BD40496C3997ceEa2b to the 0x7a4d4872Bdad98DAC4BF4276186FBD0959DAA07F account.
  2. I got the 'valid affiliate code' message.
  3. Then, I did a wrap transaction --> referral address was removed from my local storage
  4. I did several Expired transaction, then I was able to get a successful sell transaction: https://protocol-explorer.staging.gnosisdev.com/address/0x7a4d4872bdad98dac4bf4276186fbd0959daa07f
  5. Total trades section was updated accordingly.
  6. But I do not see referral info for the account 0xFF714b8b0e2700303eC912BD40496C3997ceEa2b
    image

0x7a4d4872Bdad98DAC4BF4276186FBD0959DAA07F was first referred by another user:

image

Yes, I see on UI the referral data for this account...
image

More than strange to see this message then
image

However, I might assume that this referral address was shared to the account when testing in a Dev server, as the account has a successful trade there
https://protocol-explorer.dev.gnosisdev.com/address/0x7a4d4872Bdad98DAC4BF4276186FBD0959DAA07F

Anyways, if a referral address from the Dev is valid for Prod, we should not show 'valid referral address' message

@elena-zh elena-zh added app:CowSwap CowSwap app Protofire Handled by Protofire development team Medium Severity indicator. It causes some undesirable behavior, but the system is still functional Bug Something isn't working labels Nov 17, 2021
@alfetopito
Copy link
Contributor

This is a known issue I reported last week.
I lost track of it when I went on holiday.

Digging through slack, I see that you created an issue for it, but it has not been fixed yet #1819

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
app:CowSwap CowSwap app Bug Something isn't working Medium Severity indicator. It causes some undesirable behavior, but the system is still functional Protofire Handled by Protofire development team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants