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

fix: remove pending transfer toast #92

Merged
merged 3 commits into from
Mar 22, 2022
Merged

Conversation

CarmitKl
Copy link

@CarmitKl CarmitKl commented Mar 22, 2022

Remove the call to show the toast: "Waiting for transaction to be accepted on L2..."
Keep all related code written, because soon we might want to re-display it.


This change is Reviewable

@CarmitKl CarmitKl requested a review from dan-ziv March 22, 2022 14:12
@CarmitKl CarmitKl added ready for review Pull request is ready to be reviewed and removed IN PROGRESS labels Mar 22, 2022
@dan-ziv dan-ziv changed the title fix: remove the pending transfer toast fix: remove pending transfer toast Mar 22, 2022
@CarmitKl CarmitKl merged commit 20f51f3 into dev Mar 22, 2022
@CarmitKl CarmitKl deleted the fix/removePendingTransferToast branch March 22, 2022 14:38
dan-ziv pushed a commit that referenced this pull request Mar 29, 2022
 ## [0.1.0-dev.1](v0.0.1...v0.1.0-dev.1) (2022-03-29)

### 🧩 Features

* handle browser support  ([#96](#96)) ([56d920d](56d920d))
* remove redux and use providers ([#98](#98)) ([ed38ae4](ed38ae4))
* **toasts:** add link to transfer log ([#79](#79)) ([b743b85](b743b85))
* use blockNumber=pending for balanceOf calls ([#58](#58)) ([9a84b05](9a84b05))

### 💉 Improvements

* continuous integration  ([#99](#99)) ([adc15b5](adc15b5))
* utils ([#82](#82)) ([c38e56e](c38e56e))

### 🔧 Fixes

* add missing env poll block number interval ([d364cb4](d364cb4))
* ci ([2a92c4a](2a92c4a))
* ci ([541e34b](541e34b))
* **getTransactionHash:** add fee and version to calculation ([#90](#90)) ([938ffe0](938ffe0))
* redux error on switching menu "non-serializable value was detected" ([#89](#89)) ([9dc4972](9dc4972))
* remove pending transfer toast ([#92](#92)) ([20f51f3](20f51f3))
* update ETH token addresses and add SLF token ([#84](#84)) ([2c5a9df](2c5a9df))
@dan-ziv
Copy link
Collaborator

dan-ziv commented Mar 29, 2022

🎉 This PR is included in version 0.1.0-dev.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

dan-ziv pushed a commit that referenced this pull request Apr 4, 2022
 ## [1.0.0-dev.1](v0.0.1...v1.0.0-dev.1) (2022-04-04)

### ⚠ BREAKING CHANGES

* alpha version (#126)

### 💉 Improvements

* utils ([#82](#82)) ([c38e56e](c38e56e))

### 🔧 Fixes

* add missing env poll block number interval ([d364cb4](d364cb4))
* **getTransactionHash:** add fee and version to calculation ([#90](#90)) ([938ffe0](938ffe0))
* redux error on switching menu "non-serializable value was detected" ([#89](#89)) ([9dc4972](9dc4972))
* remove pending transfer toast ([#92](#92)) ([20f51f3](20f51f3))
* update ETH token addresses and add SLF token ([#84](#84)) ([2c5a9df](2c5a9df))

### 🧩 Features

* alpha version ([#126](#126)) ([5b15ca0](5b15ca0))
* handle browser support  ([#96](#96)) ([56d920d](56d920d))
* remove redux and use providers ([#98](#98)) ([ed38ae4](ed38ae4))
* **toasts:** add link to transfer log ([#79](#79)) ([b743b85](b743b85))
* use blockNumber=pending for balanceOf calls ([#58](#58)) ([9a84b05](9a84b05))
@dan-ziv
Copy link
Collaborator

dan-ziv commented Apr 4, 2022

🎉 This PR is included in version 1.0.0-dev.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

dan-ziv pushed a commit that referenced this pull request Apr 4, 2022
 ## [1.0.0](v0.0.1...v1.0.0) (2022-04-04)

### ⚠ BREAKING CHANGES

* alpha version (#126)

### 💉 Improvements

* utils ([#82](#82)) ([c38e56e](c38e56e))

### 🧩 Features

* alpha version ([#126](#126)) ([5b15ca0](5b15ca0))
* handle browser support  ([#96](#96)) ([56d920d](56d920d))
* remove redux and use providers ([#98](#98)) ([ed38ae4](ed38ae4))
* **toasts:** add link to transfer log ([#79](#79)) ([b743b85](b743b85))
* use blockNumber=pending for balanceOf calls ([#58](#58)) ([9a84b05](9a84b05))

### 🔧 Fixes

* add missing env poll block number interval ([d364cb4](d364cb4))
* empty callback in promise .then ([#127](#127)) ([9d13c3b](9d13c3b))
* **getTransactionHash:** add fee and version to calculation ([#90](#90)) ([938ffe0](938ffe0))
* redux error on switching menu "non-serializable value was detected" ([#89](#89)) ([9dc4972](9dc4972))
* remove pending transfer toast ([#92](#92)) ([20f51f3](20f51f3))
* update ETH token addresses and add SLF token ([#84](#84)) ([2c5a9df](2c5a9df))
@dan-ziv
Copy link
Collaborator

dan-ziv commented Apr 4, 2022

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review Pull request is ready to be reviewed released on @dev released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants