Skip to content

Remove link to VCHI dashboard & restructure intro paragraphs#1391

Merged
BekahHW merged 4 commits intoVirtual-Coffee:mainfrom
adiati98:fix/remove-vchi-dashboard-link
Sep 10, 2025
Merged

Remove link to VCHI dashboard & restructure intro paragraphs#1391
BekahHW merged 4 commits intoVirtual-Coffee:mainfrom
adiati98:fix/remove-vchi-dashboard-link

Conversation

@adiati98
Copy link
Contributor

Linked Issue

Closes #1390

Description

This PR holds the below changes:

  1. Removed the link to VCHI dashboard
  2. Added info to register for VCHI by filling out form pinned in the #monthly-challenge channel on Slack
  3. Fixed nested paragraph tags to avoid hydration in the intro
  4. Restructured the order of intro paragraphs for clarity

Methodology

Code of Conduct

By submitting this pull request, you agree to follow our Code of Conduct

@adiati98 adiati98 requested a review from a team as a code owner September 10, 2025 07:27
@netlify
Copy link

netlify bot commented Sep 10, 2025

👷 Deploy Preview for virtual-coffee-io processing.

Name Link
🔨 Latest commit 45461fc
🔍 Latest deploy log https://app.netlify.com/projects/virtual-coffee-io/deploys/68c12888e7a6f800081fac2f

@adiati98 adiati98 requested a review from BekahHW September 10, 2025 07:27
@adiati98
Copy link
Contributor Author

@BekahHW I somehow can't merge this in. Can you please do that? Thanks!

@BekahHW
Copy link
Member

BekahHW commented Sep 10, 2025

@adiati98 it won't let me either. Maybe @dominicduffin1 can give it a review and we can merge it.

@adiati98
Copy link
Contributor Author

@BekahHW okay. That's kinda weird merge behavior, though 😂

@BekahHW
Copy link
Member

BekahHW commented Sep 10, 2025

@adiati98 it definitely is. I thought I was a super contributor but maybe I'm not. It looks like PRs can't be merged unless a super contributor approves. I'll try to look at it when I'm at my computer

@adiati98
Copy link
Contributor Author

@BekahHW I thought all maintainers have permission to merge in. 🤔

I think only me and @dominicduffin1 (and maybe Abbey) are super contributors. All I know is that super contributors can't review PRs that they author, but they can merge in once it's approved. But I somehow can't do it now. 😅

@BekahHW
Copy link
Member

BekahHW commented Sep 10, 2025

This is what I'm getting when I try to bypass requirements. I can't change it--at least from mobile. But it says none of these are options on this repo. If no one has changed anything, maybe it's a GH bug 🤔

image

@adiati98
Copy link
Contributor Author

I'm on mobile, too, so I can't see any settings here.
I don't think anyone has changed anything. I could still review and merge in (other than my) PRs yesterday.

Yeah, it's probably a bug. It's a so not cool one 😒

@BekahHW BekahHW merged commit ba4951b into Virtual-Coffee:main Sep 10, 2025
4 checks passed
@adiati98
Copy link
Contributor Author

@BekahHW heyyyy, it works!!!! Thanks so much! 🥳
How did you do that?

@adiati98 adiati98 deleted the fix/remove-vchi-dashboard-link branch September 10, 2025 15:51
@BekahHW
Copy link
Member

BekahHW commented Sep 10, 2025

@adiati98 i think it's a mobile bug. It worked on desktop

@adiati98
Copy link
Contributor Author

Ah, gotcha. 👍

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.

Remove link to VCHI dashboard & restructure intro paragraphs

2 participants