Skip to content

Add UI Builder link #617

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

Merged
merged 2 commits into from
Aug 11, 2025
Merged

Add UI Builder link #617

merged 2 commits into from
Aug 11, 2025

Conversation

ericglau
Copy link
Member

@ericglau ericglau commented Aug 8, 2025

No description provided.

@ericglau ericglau requested a review from pasevin August 8, 2025 19:06
@ericglau ericglau marked this pull request as ready for review August 8, 2025 19:06
@ericglau ericglau requested a review from a team as a code owner August 8, 2025 19:06
</div>

<div class="flex items-center">
<a href="http://builder.openzeppelin.com" target="_blank" rel="noopener noreferrer" class="bg-[#3d3b5d] text-white px-4 py-2 rounded-full text-sm font-medium flex items-center gap-2 hover:bg-[#4a4872] transition-colors no-underline whitespace-nowrap">
Copy link
Contributor

Choose a reason for hiding this comment

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

@pasevin would be interesting to pass the network in the URL (would save 1 step in the builder and directly be at contract step)

Copy link
Member Author

Choose a reason for hiding this comment

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

With the Solidity Wizard, we don't know which EVM network the user is using. This would only be applicable for cases where the language and network is 1-1.

<div class="flex flex-row gap-2">
<div class="text-sm text-gray-600 leading-tight text-right">
<div>After deploying a contract, use the new</div>
<div>Contracts UI Builder to spin up a quick UI</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: "spin up a quick UI" is a bit unclear to me what the "UI builder" would do, maybe keywords like "transaction" "contract interaction" would be clearer

Copy link
Member Author

Choose a reason for hiding this comment

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

This directs the user's attention, and the actual text in https://builder.openzeppelin.com/ provides more context.

@ericglau ericglau requested a review from CoveMB August 11, 2025 13:15
Copy link

@pasevin pasevin left a comment

Choose a reason for hiding this comment

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

lgtm, thanks! Hopefully a good start for a great friendship between the wizard and the builder ;)

@ericglau ericglau merged commit 374d325 into OpenZeppelin:master Aug 11, 2025
17 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Aug 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants