Skip to content

Conversation

@Legend-Master
Copy link
Contributor

Description

@netlify
Copy link

netlify bot commented Apr 15, 2025

Deploy Preview for tauri-v1 ready!

Name Link
🔨 Latest commit ddeb793
🔍 Latest deploy log https://app.netlify.com/sites/tauri-v1/deploys/67fe882f38cf1400083e04ed
😎 Deploy Preview https://deploy-preview-3259--tauri-v1.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Legend-Master
Copy link
Contributor Author

@RazielMartelus96 do you think this is good enough?

image

@RazielMartelus96
Copy link

Looks good to me, personally would go for error red, but I am a glutton for making things completely obvious so I will chop that up to personal preference haha

@Legend-Master Legend-Master marked this pull request as ready for review April 15, 2025 15:11
@Legend-Master Legend-Master requested a review from a team as a code owner April 15, 2025 15:11
@vasfvitor
Copy link
Contributor

vasfvitor commented Apr 15, 2025

image
I'd go with danger too @Legend-Master I went ahead and changed it, lmk if that's ok and I'll merge.

yellow still fits the website tint I think

@Legend-Master
Copy link
Contributor Author

I'm good with either one, the danger one definitely pops a lot more

@vasfvitor vasfvitor merged commit 7fea48a into v1 Apr 15, 2025
7 checks passed
@vasfvitor vasfvitor deleted the announcement-warning-color branch April 15, 2025 17:06
@github-project-automation github-project-automation bot moved this from 🪵 Backlog to ✅ Done in Documentation Apr 15, 2025
@RazielMartelus96
Copy link

Thank you kindly one and all. I know it may seem like a little change, but I have a feeling this will lead to a small uptick in devs actually getting past the initial setup phase! <3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

4 participants