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

Release 1.2.0 #1341

Merged
merged 46 commits into from
Dec 15, 2022
Merged

Release 1.2.0 #1341

merged 46 commits into from
Dec 15, 2022

Conversation

katspaugh
Copy link
Member

@katspaugh katspaugh commented Dec 9, 2022

Features

Bug fixes

Chores

germartinez and others added 30 commits November 29, 2022 08:41
* Added sandbox attribute to Safe App iframe

* added allow-popups-to-escape-sandbox
* Chore: add a QR code to PR deployment comment

* Add a pre-deploy comment

* Message id

* ref_name

* Branch name

* Replace slashes

* add-pr-comment@v2

* head_ref
* fix: Account switching with mobile pairing and WC

* chore: Add TODO
* Chore: add a CLA checking action

* Rm myself to test

* Refs

* PR trigger

* Tweak the if

* Restore myself
* Chore: tune the CLA action

* pull_request_target -> pull_request

* Access token

* Rm optional params

* Lock after merge

* Copy CLA into an md file
* Fix: link to old app from the import page

* path-to-signatures

* CLA contains

* Revert conditional alert
* fix: Handle timeout error in safe creation

* fix: Failing tests, adjust isContractError

* chore: Add TODO
* feat: replace specific transactions (#1053)

* feat: replace specific transactions

* fix: cleanup code

* fix: update title

* style: change `variant`

* fix: replacing `0` nonced transactions

* feat: add autocomplete to `NonceForm` nonce (#1059)

* feat: replacement modal (#1132)

* feat: replace transaction modal w/ stepper

* fix: adjust text + padding/margins

* fix: adjust alignment + add mobile view

* fix: alignment

* fix: tweak autocomplete item text

* fix: shorten text + add `Backdrop`

* Merge branch 'dev' into tx-replacement

* fix: replace nonce 0, flicker, no spending limit + disable duplite rejections (#1300)

* fix: replace nonce 0, flicker + no spending limit

* fix: always allow replacement + nonce `0`

* fix: remove tag

* Rm backdrop

Co-authored-by: katspaugh <[email protected]>

Co-authored-by: katspaugh <[email protected]>
* Chore: replace the CLA action with a simple comment

* Test the comment

* Detect signature comment

* Trigger on comment
* chore: fix vertical scrolling after opening the Mobile paring modal

* cheking CLA

* alternative fix: make Dialog a controlled component

Co-authored-by: katspaugh <[email protected]>
* fix: blur number inputs `onWheel` events

* fix: remove duplicate attribute

* fix: use `inputMode` and `pattern`

* fix: use `type` attribute + replace more fields

* fix: missing field

* fix: remove attribute
* Chore: try a different CLA config

* Update allowlist
* fix: format review amount to locale

* add truncating trailing zeros to the formater. tests

* use toLocaleString instead of regex to format the integer part of amount

* use the common formatter for token amounts in the review modal heasder

* precision formatter without using the style guide

* add tests to new formatter

* rm redundant code
* fix: Only initialize the Core SDK once

* fix: Hide Send button in Address book if wrong network

* fix: Add tests for useInitSafeCoreSDK hook

* refactor: useInitSafeCoreSDK hook

* refactor: Rename initSafeSDK param

* fix: Reset core sdk when switching safes

* fix: Add safeSDK dependency for safe app modals

* Refactor: useTxSender

* fix: Adjust imports

* refactor: Use useTxSender hook in components

* fix: Handle undefined values in SignOrExecuteForm

* refactor: Extract spending limit tx to the tx sender

* fix: Failing tests

* fix: Add useTxSender functions to dependency arrays

* fix: Better type useTxSender hook

* fix: Remove ts-ignore

* fix: Remove async from undefined function

Co-authored-by: katspaugh <[email protected]>
Bumps [decode-uri-component](https://github.com/SamVerschueren/decode-uri-component) from 0.2.0 to 0.2.2.
- [Release notes](https://github.com/SamVerschueren/decode-uri-component/releases)
- [Commits](SamVerschueren/decode-uri-component@v0.2.0...v0.2.2)

---
updated-dependencies:
- dependency-name: decode-uri-component
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@github-actions
Copy link

github-actions bot commented Dec 9, 2022

Branch preview

✅ Deploy successful!

https://release120--webcore.review-web-core.5afe.dev

@github-actions
Copy link

github-actions bot commented Dec 9, 2022

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 0 0
Ignored 0 N/A
  • Result: ✅ success
  • Annotations: 0 total

Report generated by eslint-plus-action

@katspaugh
Copy link
Member Author

As @schmanu mentioned, we need to release the Claiming App before we release this.

iamacook and others added 2 commits December 12, 2022 10:37
* fix: adjust text of buttons in transfer modal

* fix: adjust text

* Update src/components/tx/modals/NewTxModal/ReplacementModal.tsx

Co-authored-by: katspaugh <[email protected]>
@liliya-soroka
Copy link
Member

liliya-soroka commented Dec 12, 2022

Regression issue = #1353

  1. Text is displayed inside amount field
    -Go to the send funds
  • enter 1. to the amount field

image

image

@liliya-soroka
Copy link
Member

liliya-soroka commented Dec 12, 2022

Regression issue - #1354
2. module tx is not fixed fully - fix: fallback to "Contract interaction" for module (#1301)
Settings icon is used instead of contract interaction
image

@JagoFigueroa
Copy link

JagoFigueroa commented Dec 12, 2022

Regression issue - #1355
3. Hola team! For this ticket regarding timeouts on safe creation the expected result is shown to me after 6,5 minutes on the case that I reload the page:

Screenshot 2022-12-12 at 17 17 55

but unfortunately on the case that I don't perform a manual page reload it will be stuck on this screen forever (I have waited for 20 minutes in chrome)

Screenshot 2022-12-12 at 17 05 01

@JagoFigueroa
Copy link

JagoFigueroa commented Dec 13, 2022

Regression issue = #1356
4. Hola again! For this señor ticket regarding the export/import for this case mentioned on the ticket:

  • Show the "Overwrite data?" alert only after dropping a file

I can see that we are showing the alert before uploading the JSON file. Nothing major or anything but Is it expected?

Screenshot 2022-12-13 at 09 59 35

Cheers!

@JagoFigueroa
Copy link

JagoFigueroa commented Dec 13, 2022

Regression issue = #1357
5. Regarding transaction replacement modal, presentation is a bit off when viewing it from a mobile device (video is from the browser but in mobile resolution I was able to reproduce this as well on my Iphone 13 pro):

replace.mobile.mp4

iamacook and others added 3 commits December 13, 2022 13:01
* fix: handle nullable `SafeInfo['version']

* fix: install + fix types w/ new package

* fix: remove unnecessary optional chaining

* fix: add fallback version text

* fix: don't show link for unsupported contract
@liliya-soroka
Copy link
Member

The 1.2.0 is approved for the release without - #1355, but we are waiting for the critical issue #1343 to be fixed, tested and included into 1.2.0

iamacook and others added 2 commits December 14, 2022 13:37
* fix: take native number validation into account

* fix: remove native validation + revert to text
@liliya-soroka
Copy link
Member

liliya-soroka commented Dec 14, 2022

Thank you for the fixes. The 1.2.0 rc is approved to go live

@katspaugh katspaugh merged commit d892513 into main Dec 15, 2022
@katspaugh katspaugh deleted the release/1.2.0 branch December 15, 2022 09:13
@github-actions github-actions bot locked and limited conversation to collaborators Dec 15, 2022
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.