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

Update hackclub.com.yaml #1452

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

BennyGaming635
Copy link

[Adding/Deleting/Updating] subdomain.[hackclub.com/dino.icu/etc]

Description

Provide a description here of the changes, and if adding a subdomain provide a rationale & what it will be used for.

@BennyGaming635 BennyGaming635 requested review from a team as code owners December 12, 2024 00:15
@BennyGaming635
Copy link
Author

BennyGaming635 commented Dec 12, 2024

Adding this subdomain as we're a new hack club that has had their meeting with jps. Used to advertise our group and what we do. Also anything regarding our Hack Club.

Copy link
Member

@alx-alexpark alx-alexpark left a comment

Choose a reason for hiding this comment

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

Hii!

Thanks for submitting your PR. Unfortunately, there are a few issues that need to be resolved.

Firstly, you should replace SUBDOMAIN_NAME with the subdomain that you want, like this.

henleyhigh:
  - ttl: 600
    type: CNAME
    value: <another domain name based on where you are hosting your site>

The value of the CNAME is essentially a reference to the server that is hosting your website. I can't tell you what to put here, as it depends on where and how you are hosting your website.

Finally, after you do so, please make sure to sort the record within the file so that it is in the appropriate alphabetical order.

Thanks! and feel free to DM me on slack (@alex (p variant)) if you need any help.

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.

2 participants