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

Apps: update for @aragon/rpc-messenger's new deferred behaviour #871

Merged
merged 1 commit into from May 29, 2019

Conversation

sohkai
Copy link
Contributor

@sohkai sohkai commented May 27, 2019

Breaking change from aragon/aragon.js#305 (to be released as @aragon/rpc-messenger@2).

I believe intents were the only RPC requests we were sending without caring for their response. All other requests were either made for their response, or we wanted to have feedback if the user cancelled (e.g. identity modifications)

@sohkai sohkai requested a review from 2color May 27, 2019 23:23
@luisivan luisivan mentioned this pull request May 27, 2019
25 tasks
@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.015% when pulling f9edda9 on update-deferred-api into 7bfbb52 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.015% when pulling f9edda9 on update-deferred-api into 7bfbb52 on master.

Copy link
Contributor

@2color 2color left a comment

Choose a reason for hiding this comment

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

🔥

@sohkai sohkai changed the title Apps: update for @aragon/rpc-messenger's deferred behaviour Apps: update for @aragon/rpc-messenger's new deferred behaviour May 29, 2019
@sohkai sohkai merged commit 9c2a422 into master May 29, 2019
@sohkai sohkai deleted the update-deferred-api branch May 29, 2019 10:36
2color added a commit that referenced this pull request May 29, 2019
…ng-state

* origin/master:
  Apps: update for @aragon/rpc-messenger's new deferred behaviour (#871)
  Travis: optimizations and refactor to use build stages (#865)
  Update Travis CI badge (#866)
  Hoist common dependencies (#863)
2color added a commit that referenced this pull request May 29, 2019
…t-api

* token-manager-loading-state:
  Upgrade @aragon/ui to 0.40.1
  Apps: update for @aragon/rpc-messenger's new deferred behaviour (#871)
  Use a boolean for loading indicator
  Token Manager: update @aragon/api dependencies
  Token Manager: fix lint
  Token Manager: only use the SyncIndicator for the loading states (#873)
  Token Manager: fix loading indicator not being shown when syncing
  Upgrade aragon ui
  Handle custom events before contract events
  Remove unneeded import
  Update apps/token-manager/app/src/script.js
  Travis: optimizations and refactor to use build stages (#865)
  Update Travis CI badge (#866)
  Hoist common dependencies (#863)
  Reduce sync status in background script
  Add syncing states (faked for now)
MickdeGraaf pushed a commit to MickdeGraaf/aragon-apps that referenced this pull request Jan 28, 2020
ramilexe pushed a commit to ConsiderItDone/aragon-apps that referenced this pull request Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants