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

Feature/refactor #195

Open
wants to merge 975 commits into
base: main
Choose a base branch
from

Conversation

skylinezum
Copy link

Ensure that yarn@1.x can be executed. and fix Bugs under examples

Co-Authored-By: @btcSteven 648800939@qq.com

@@ -179,8 +181,9 @@ export const GrantModal = ({ isOpen, onClose, chainName }: GrantModalProps) => {
onClick={() => {}}
/>
</PopoverTrigger>
<Box></Box>
Copy link

Choose a reason for hiding this comment

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

what is this box for?

Choose a reason for hiding this comment

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

It's probably a mistake write, I 'll fixed it

@sdqede
Copy link

sdqede commented Aug 13, 2024

Phatg also mentioned that when using yarn1, there will be an error "Cannot read properties of null (reading 'useRef')", I tried to build the examples/nft, and encountered this error
so you might need to check those examples and make them built successfully, otherwise, all GitHub actions will fail

@btcSteven
Copy link

okay ,I fixed it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants