Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
loki344 pushed 1 commit to feat(frontend)/enhance-dapp-description dfinity/oisy-wallet
- feat(frontend): extend existing interface to extract oisy specific fields 2d6bb3c
peterpeterparker pushed 1 commit to feat/copy-block-receive-address dfinity/oisy-wallet
- feat: spacing and text for later fe2eb07
DenysKarmazynDFINITY created a branch on dfinity/oisy-wallet
feat(frontend)/carousel-component - A browser-based, multi-chain wallet hosted on the Internet Computer
peterpeterparker pushed 1 commit to feat/copy-block-receive-address dfinity/oisy-wallet
- feat: display or not qr code button cb9c1f8
peterpeterparker created a comment on a pull request on dfinity/oisy-wallet
Feels like we should bump Svelte v5 so that you resolve this with the new `Snippet` pattern. Do you have a WIP branch to share in which those utils are used? Just to grasp a better sense of how ...
darioloetsc created a comment on a pull request on dfinity/oisy-wallet
will split in two, and refactoring is bigger than expected
darioloetsc closed a pull request on dfinity/oisy-wallet
feat(frontend): add token group store
# Motivation Introduce a token group store to preserve expanded settings in the grouping view. # Changes - Add token group store # Tests
DenysKarmazynDFINITY created a comment on a pull request on dfinity/oisy-wallet
@peterpeterparker also a valid question, and it was my initial intention to do so, but it appeared it's not possible to work with `slots` in Svelte similar to `React.children`: looping through them...
peterpeterparker created a comment on a pull request on dfinity/oisy-wallet
Thanks for the explanation. I then have a follow-up question: why manipulating the DOM manually and not resolving the implementation using Svelte? Fortunatelly Svelte has not virtual DOM but, I'...
peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
same question for usd
peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
Does it really matter here? It's all dollars, shouldn't the array just be reduced to sum all usd balance?
peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
what if token2 balance is zero but token1 balance is >0 ?
peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
- Should we add a guard if there are more than two tokens? - If exactly two tokens, should the array be actually typed as `[TokenUi, TokenUi]`?
peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
Maybe I'm misleading it but,`null` is maybe not specified as return type of the function?
peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
definitely another PR, thanks
DenysKarmazynDFINITY created a comment on a pull request on dfinity/oisy-wallet
@peterpeterparker yes, the CSS approach was my first thought, but, unfortunately, it doesn't align with the requirements for the wanted carousel component. To name a few reasons why we do need to m...
peterpeterparker pushed 1 commit to feat/copy-block-receive-address dfinity/oisy-wallet
- fix: merge error 04320ef
AntonioVentilii-DFINITY created a review comment on a pull request on dfinity/oisy-wallet
right! make sense! can i do it in another PR? because I am pretty sure it will change some other components and should be indeed a different topic
AntonioVentilii-DFINITY pushed 1 commit to feat(frontend)/Include-balances-in-token-group dfinity/oisy-wallet
- chore: adjust string fdf27b7
AntonioVentilii-DFINITY created a review comment on a pull request on dfinity/oisy-wallet
- done! and created test - good catch! I completely missed that... - see first point please review it again, because i separated the smaller functions and created tests about it
peterpeterparker pushed 3 commits to feat/copy-block-receive-address 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/copy-block-receive-address 35906e0
peterpeterparker pushed 1 commit to main dfinity/oisy-wallet
- feat(frontend): receive modal title with symbol (#3036) # Motivation The designers want to display the token symb... 31612ac
peterpeterparker closed a pull request on dfinity/oisy-wallet
feat(frontend): receive modal title with symbol
# Motivation The designers want to display the token symbol within the "Receive" modal title. # Changes - Since `addressToken` is per definition potentially `undefined` either just display...