Usame Algan

Results 31 comments of Usame Algan

Is this the error when trying to execute the transaction via relay? Afaik we don't allow transactions from a safe to itself in our relayer config. Otherwise it usually works...

Seems like the address is truncated at the end now. @liliya-soroka is this ok enough to close the issue? ![Image](https://github.com/safe-global/safe-wallet-web/assets/5880855/92975d7c-0830-40f8-9e81-859b409b0e80)

Did some research and [discussion](https://5afe.slack.com/archives/C049H57FS1E/p1681480040397569) with the team. TLDR: - The default `GITHUB_TOKEN` we use doesn't have enough permissions on forked (external) PRs for [security reasons](https://securitylab.github.com/research/github-actions-preventing-pwn-requests/) - We can rewrite...

I suggest we only clean up the tests with this lint check. We often use `await act` when its not necessary or `await findByRole` instead of just `getByRole`. We can...

I think ideally we would only have one list of transactions and just switch their status to pending and remove all labels like Next/Queued and only have one label at...

I can also reproduce this on app.safe.global but not on localhost. Also tried to run it locally with `build` and `static-serve` without success. I can somewhat "force" the issue by...

After some more investigation I think this issue happens only if WC is really slow. I am not sure yet what leads to that but I think we should try...

This fixes the issue mentioned here https://github.com/cowprotocol/cowswap/pull/4063#issuecomment-2039588336

I will merge this so the CoW team is unblocked but we should follow up on this once Metamask works again.