Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

dfinity/oisy-wallet

peterpeterparker created a review on a pull request on dfinity/oisy-wallet

View on GitHub

loki344 created a comment on a pull request on dfinity/oisy-wallet
maybe take a look at: https://svelte.dev/playground/c379be902322415bb1e10f855442f40a?version=4.2.1 Seems to use transform magic

View on GitHub

AntonioVentilii-DFINITY created a review comment on a pull request on dfinity/oisy-wallet
i wanted to do it at usage level: you mean to set the logic here right? If `loading` then `disabled` for sure?

View on GitHub

AntonioVentilii-DFINITY created a review on a pull request on dfinity/oisy-wallet

View on GitHub

peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
btw. can a button be set as disabled and loading at the same time?

View on GitHub

peterpeterparker created a review on a pull request on dfinity/oisy-wallet

View on GitHub

AntonioVentilii-DFINITY created a review comment on a pull request on dfinity/oisy-wallet
Good question... The only issue I see is to keep it consistent with definition of `TokenUi.usdBalance` that can be either `number` or `undefined`, but not `null`. When it is `undefined`, we s...

View on GitHub

AntonioVentilii-DFINITY created a review on a pull request on dfinity/oisy-wallet

View on GitHub

peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
For consistency with `sumTokenBalances`, shouldn't `sumTokenUsdBalances` returns `null` then if both tokens are `undefined`? Or should the function always returns a zero or value, hence suggesti...

View on GitHub

peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
I oversaw it, good point.

View on GitHub

peterpeterparker created a review on a pull request on dfinity/oisy-wallet

View on GitHub

peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
Yes, works for me.

View on GitHub

peterpeterparker created a review on a pull request on dfinity/oisy-wallet

View on GitHub

peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
Nvm my comment, my bad.

View on GitHub

peterpeterparker created a review on a pull request on dfinity/oisy-wallet

View on GitHub

AntonioVentilii-DFINITY created a review on a pull request on dfinity/oisy-wallet
LGTM! Tks

View on GitHub

AntonioVentilii-DFINITY created a review on a pull request on dfinity/oisy-wallet
I think there was a misunderstanding: your first iteration was ok, I would have just used the `initCertifiedSetterStore` So, that the definition would have been ```typescript export interfac...

View on GitHub

loki344 created a review comment on a pull request on dfinity/oisy-wallet
Sorry ,I don't understand your question.

View on GitHub

loki344 created a review on a pull request on dfinity/oisy-wallet

View on GitHub

loki344 pushed 4 commits to feat(frontend)/enhance-dapp-modal dfinity/oisy-wallet
  • feat(frontend): network icon black white for receive (#3064) # Motivation The network icon displayed in the recei... 9fb3226
  • feat(frontend): receive modal title with symbol (#3036) # Motivation The designers want to display the token symb... 31612ac
  • feat(frontend): enhance dapp data (#3063) # Motivation The Design-Team added more social links and new images for... 885772a
  • Merge remote-tracking branch 'origin/main' into feat(frontend)/enhance-dapp-modal # Conflicts: # src/frontend/src/li... b9df8bc

View on GitHub

loki344 pushed 1 commit to main dfinity/oisy-wallet
  • feat(frontend): enhance dapp data (#3063) # Motivation The Design-Team added more social links and new images for... 885772a

View on GitHub

loki344 deleted a branch dfinity/oisy-wallet

feat(frontend)/enhance-dapp-description

loki344 closed a pull request on dfinity/oisy-wallet
feat(frontend): enhance dapp data
# Motivation The Design-Team added more social links and new images for the dapp-explorer. This MR updates adds the new data. # Changes - Add images for decideID and OpenChat - New image ...
loki344 pushed 3 commits to feat(frontend)/enhance-dapp-description dfinity/oisy-wallet
  • feat(frontend): network icon black white for receive (#3064) # Motivation The network icon displayed in the recei... 9fb3226
  • feat(frontend): receive modal title with symbol (#3036) # Motivation The designers want to display the token symb... 31612ac
  • Merge branch 'main' into feat(frontend)/enhance-dapp-description 77c5e4e

View on GitHub

peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
Cool

View on GitHub

peterpeterparker created a review on a pull request on dfinity/oisy-wallet

View on GitHub

peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
not for this PR but, I guess this can become `true` actually?

View on GitHub

peterpeterparker created a review on a pull request on dfinity/oisy-wallet
LGTM, thx

View on GitHub

peterpeterparker created a review on a pull request on dfinity/oisy-wallet
LGTM, thx

View on GitHub

AntonioVentilii-DFINITY opened a pull request on dfinity/oisy-wallet
feat(frontend): Add loading style to Button component
# Motivation We want to display "loading" buttons in the Hero, similar to a Skeleton, but resembling the pulsating effect of the balance on loading. So, as first step, we add `loading` prop t...
Load more