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.

Sign up for GitHub

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

Jump to bottom

feat(nfts): ui-lib integration into taikoon-ui #17220

Merged
merged 48 commits into from
May 22, 2024
Merged

feat(nfts): ui-lib integration into taikoon-ui #17220

merged 48 commits into from
May 22, 2024

Conversation

bearni95
Copy link
Contributor

@bearni95 bearni95 commented May 16, 2024 •
edited
Loading

taikoon-ui integration of ui-lib's footer component

bearni95 and others added 30 commits May 6, 2024 17:02
@bearni95
created svelte lib package
8e43906
@bearni95
storybook scaffolding
a044235
@bearni95
storybook cleanup
2d75386
@bearni95
dirty but functional scaffolding on storybook + svelte + tailwind
b5f64f2
@bearni95
lib export working properly
943ab16
@bearni95
test install for lib in taikoon-ui
63e3f31
@bearni95
adding missing commit files
5803954
@bearni95
ui lib packaged + footer
a019d04
@bearni95
readme
13f593a
@bearni95
added link onto svelte+storybook docs
02288e6
@bearni95
Merge branch 'main' into ui-lib
1d25e9a
@bearni95 @github-actions
forge fmt & update contract layout table
82158c3
@bearni95 @github-actions
forge fmt & update contract layout table
2c4c8b9
@bearni95
fixed fonts on storybook
3399cd1
@bearni95
removed autodocs from storybook
b98020f
@bearni95
restored configs for storybook constrols
9949571
@bearni95
Merge branch 'ui-lib' of ssh://github.com/taikoxyz/taiko-mono into ui…
6e4dfd4 
…-lib
@bearni95
refreshed pnpm-lock.yaml
be33a01
@bearni95
Merge branch 'main' into ui-lib
bc5b2b2
@bearni95
example export/import of lib's tailwind config; fully ported footer o…
5b79b7c 
…n taikoon-ui from ui-lib
@bearni95
support for light/dark theme switch
d2e5269
@bearni95
scaffolding for color theme exports
ad36303
@bearni95
non-theme colors imported
103b9e7
@bearni95
colors and themes included
9d74253
@bearni95
updated pnpm-lock.yml
4c97906
@bearni95
Merge branch 'main' into ui-lib
0c33637
@bearni95
revert docs involuntary changes
5c337bb
@bearni95
reverted protocol changes
5cda3ad
@bearni95
reverting more lefthook auto changes
dfd82df
@bearni95
Merge branch 'main' into ui-lib
14e89b2
@bearni95 bearni95 changed the title feat(taikoon): ui-lib integration into taikoo-ui feat(taikoon): ui-lib integration into taikoon-ui May 16, 2024
@dantaik dantaik changed the title feat(taikoon): ui-lib integration into taikoon-ui feat(taikoon-ui): ui-lib integration into taikoon-ui May 20, 2024
@bearni95 bearni95 changed the title feat(taikoon-ui): ui-lib integration into taikoon-ui feat(nfts): ui-lib integration into taikoon-ui May 20, 2024
@bearni95 bearni95 mentioned this pull request May 20, 2024
Merged
5 tasks
@bearni95 bearni95 marked this pull request as ready for review May 20, 2024 17:35
@dionysuzx
Copy link
Collaborator

dionysuzx commented May 21, 2024 •
edited
Loading

@bearni95 i think you need to update your scope in the PR title

update: i did it for you

@dionysuzx dionysuzx changed the title feat(nfts): ui-lib integration into taikoon-ui feat(taikoon-ui): ui-lib integration into taikoon-ui May 21, 2024
@bearni95 bearni95 changed the title feat(taikoon-ui): ui-lib integration into taikoon-ui feat(nfts): ui-lib integration into taikoon-ui May 21, 2024
@bearni95
Copy link
Contributor Author

@bearni95 i think you need to update your scope in the PR title

update: i did it for you

We removed taikoon as a valid title index, using nfts now as it's also the package that contains both Taikoons and Snaefell nfts

@bearni95 bearni95 requested review from dantaik and KorbinianK May 22, 2024 10:39
@bearni95 bearni95 added this pull request to the merge queue May 22, 2024
Merged via the queue into main with commit 849a3c4 May 22, 2024
7 checks passed
@bearni95 bearni95 deleted the ui-lib-taikoon-ui-integration branch May 22, 2024 10:47
@taiko-kitty taiko-kitty mentioned this pull request May 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dionysuzx dionysuzx dionysuzx approved these changes

@2manslkh 2manslkh 2manslkh approved these changes

@bennettyong bennettyong Awaiting requested review from bennettyong

@dantaik dantaik Awaiting requested review from dantaik

@KorbinianK KorbinianK Awaiting requested review from KorbinianK

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bearni95 @dionysuzx @2manslkh