Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
peterpeterparker created a review on a pull request on dfinity/oisy-wallet
LGTM, thx Please have a look at my comment before merge.
loki344 created a branch on dfinity/oisy-wallet
feat(frontend)/add-dapp-explorer-link-to-mobile-nav - A browser-based, multi-chain wallet hosted on the Internet Computer
DenysKarmazynDFINITY created a review comment on a pull request on dfinity/oisy-wallet
True, I changed the default value for `styleClass` to `''` instead of `undefined`
DenysKarmazynDFINITY pushed 1 commit to feat(frontend)/carousel-controls dfinity/oisy-wallet
- feat(frontend): review adjustments d5ce9f4
peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
`styleClass ?? ""` otherwise are we maybe ending with a class name "undefined" in the DOM?
peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
`active:text-blue-ribbon` too?
DenysKarmazynDFINITY pushed 1 commit to feat(frontend)/carousel-controls dfinity/oisy-wallet
- feat(frontend): review adjustments e292419
DenysKarmazynDFINITY created a review comment on a pull request on dfinity/oisy-wallet
I definitely do 🙂 Adjusted.
DenysKarmazynDFINITY created a review comment on a pull request on dfinity/oisy-wallet
Makes sense, create a new button component as suggested.
DenysKarmazynDFINITY pushed 1 commit to feat(frontend)/carousel-controls dfinity/oisy-wallet
- feat(frontend): review adjustments 57b0188
inc-man pushed 1 commit to main dfinity/oisy-wallet
- build(ci): automated release process (#3027) # Motivation The previously merged release process had its flaws, so... 46ffd94
inc-man closed a pull request on dfinity/oisy-wallet
build(ci): automated release process
# Motivation The previously merged release process had its flaws, so we want an new version. # Changes fixed snytax errors adjusted release-drafter configuration better formatting
peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
`on:click={onPreviousClick}` would do the job if you want to spare some characters
peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
What about creating a component `ButtonControl`? We generally do not have a pattern in which we assign classes as const.
peterpeterparker created a review comment on a pull request on dfinity/oisy-wallet
What do you think of removing the suffix `Click` for a bit less verbosity?
inc-man pushed 3 commits to build(ci)/automate-release-process dfinity/oisy-wallet
- feat(frontend): add dapp-explorer nav item (#3026) # Motivation Add navigation item to make the dapp-explorer ava... 5ea1f91
- chore(frontend): extract submit dapp btn into own component (#3045) # Motivation Extract component for submitting... d5a15ae
- Merge branch 'main' into build(ci)/automate-release-process 4a61dd9
loki344 pushed 1 commit to main dfinity/oisy-wallet
- chore(frontend): extract submit dapp btn into own component (#3045) # Motivation Extract component for submitting... d5a15ae
loki344 closed a pull request on dfinity/oisy-wallet
chore(frontend): extract submit dapp btn into own component
# Motivation Extract component for submitting a dapp to be displayed in the dapp-explorer.
loki344 pushed 2 commits to chore(frontend)/make-component-for-submit-dapp-btn dfinity/oisy-wallet
loki344 pushed 1 commit to main dfinity/oisy-wallet
- feat(frontend): add dapp-explorer nav item (#3026) # Motivation Add navigation item to make the dapp-explorer ava... 5ea1f91