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

Added Ariz button #483

Merged
merged 10 commits into from
Jul 23, 2024
Merged

Conversation

Jikugodwill
Copy link
Collaborator

@Jikugodwill Jikugodwill commented Jul 18, 2024

Screen.Recording.2024-07-19.at.10.36.13.AM.mov

Copy link

vercel bot commented Jul 18, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
gateway ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 23, 2024 4:21pm

@elliotBraem
Copy link
Contributor

elliotBraem commented Jul 18, 2024

Also, I think it looks decent where you put it, although it might be nice if in the top right (but if you're not convinced, no problem, leave it where it is)

And if you could please add tests:

"should show ariz portfolio button when owner of project"
"should navigate to ariz when clicked"

@@ -6,6 +6,5 @@
"gateway_url": "https://nearbuilders.org",
"new": "builddao.near",
"old": "buildhub.near",
"potlock": "potlock.near",
"every": "every.near"
"potlock": "potlock.near"
Copy link
Collaborator

Choose a reason for hiding this comment

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

these changes are already on develop if I remember correctly?

@elliotBraem elliotBraem force-pushed the 475-open-ariz-portfolio-on-project-page branch from c028a4c to 5651cce Compare July 23, 2024 16:06
@elliotBraem elliotBraem merged commit 67e7cba into develop Jul 23, 2024
3 of 4 checks passed
@elliotBraem elliotBraem deleted the 475-open-ariz-portfolio-on-project-page branch July 23, 2024 16:22
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.

"Open Ariz Portfolio" on project page
3 participants