Skip to content
This repository has been archived by the owner on Nov 10, 2023. It is now read-only.

Release v3.11.0 #2587

Merged
merged 21 commits into from
Aug 2, 2021
Merged

Release v3.11.0 #2587

merged 21 commits into from
Aug 2, 2021

Conversation

dasanra
Copy link
Collaborator

@dasanra dasanra commented Jul 28, 2021

Features

Fixes

Chores

fmrsabino and others added 19 commits July 16, 2021 13:41
* Add test action to launch e2e tests

* Add name to github job

* Update event-type name

* Run e2e after rinkeby build on dev
* Add centralize route proposal for settings

* Use centralize routes for address book and transactions

* Set Apps route and path in root route components

* Use makeEntryItem function to avoid code repetition on useSidebarItems
* remove safeAppsUrl, safeRelayServiceUrl, fix ethgasstation for mainnet

* fix tests
* Add separate property for safe apps rpc url

* fix polygon rpc url

* add safe apps rpc token env var to ci

* bring ethgasstaction api key back

* change infura project id env var name
* Added github action

* Added action to comment on e2e success

* Check e2e test status to comment success or fail
* Use Menu and Breadcrumb components in Balances, Transactions, Apps and Settings

* Clean unused styles on settings
* Upgrade sentry dependencies

* Upgrade bnc-onboard and dependencies with minor changes
* use image fetching func that is not affected by cors

* use arrow functions
* Use gateway-sdk for safe info and balances

* Collectibles

* Replace fiat currencies endpoint

* Load transaction history via the SDK

* Add transaction queue

* Set released version from safe-react-gateway-sdk

Co-authored-by: Daniel Sanchez <daniel.sanchez@gnosis.pm>
@github-actions
Copy link

github-actions bot commented Jul 28, 2021

CLA Assistant Lite All Contributors have signed the CLA.

@github-actions
Copy link

github-actions bot commented Jul 28, 2021

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type Occurrences Fixable
Errors 0 0
Warnings 5 0
Ignored 15 N/A
  • Result: ✅ success

  • Annotations: 5 total


[warning] @typescript-eslint/explicit-module-boundary-types

Require explicit return and argument types on exported functions' and classes' public class methods


Report generated by eslint-plus-action

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@francovenica
Copy link
Contributor

francovenica commented Jul 30, 2021

Major issue: #2591

@dasanra
Copy link
Collaborator Author

dasanra commented Jul 30, 2021

@francovenica so unexpected 😉 already patched 👌

@francovenica
Copy link
Contributor

The release branch looks fine.
What it was possible to test functionally in the ticket description I've checked already
Made a regression in rinkeby and it worked fine
the issue with the "Remove safe" button was fixed
Tested BSC and worked fine
Tested Poly and I had some issues with gas estimation (sometimes it would take a few tries to finally do the estimation) but I attribute that to the issues we have seen later this week
Checked the google analytics while doing the testing, making sure it shows in the live feed whenever I was navigating.

Looks good to me

@dasanra dasanra requested a review from katspaugh August 2, 2021 10:23
Copy link
Member

@katspaugh katspaugh left a comment

Choose a reason for hiding this comment

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

⭐️

@github-actions
Copy link

github-actions bot commented Aug 2, 2021

E2E Tests Failed

@dasanra dasanra merged commit 28b8662 into main Aug 2, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Aug 2, 2021
@katspaugh katspaugh deleted the release/v3.11.0 branch August 3, 2021 13:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants