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

Create *oli-mebberson.is-cool.dev.json #1967

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

olii-dev
Copy link

Requirements

  • You have completed your website.
  • The website is reachable.
  • The CNAME record doesn't contain https:// or /.
  • There is sufficient information at the owner field.
  • There is no NS Records (Enforced as of Sepetember 4th, 2024)

Description

I will be using the domain for my portfolio website which is personal

Link to Website

https://github.com/olii-dev/aboutme

@olii-dev olii-dev requested a review from a team as a code owner January 25, 2025 21:42
@open-domains open-domains bot added the CLA Signed Contributor has signed the CLA label Jan 25, 2025
Copy link
Member

Choose a reason for hiding this comment

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

Invalid file. Wildcards are not done like this.

Copy link
Author

Choose a reason for hiding this comment

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

How do I do them then? I followed the instructions.

Copy link
Author

Choose a reason for hiding this comment

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

@DEV-DIBSTER I just fixed all the issues i could think of that are possible.

Copy link
Member

Choose a reason for hiding this comment

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

Is there a reason for wildcard records?

Copy link
Author

Choose a reason for hiding this comment

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

I don't understand?

Copy link
Member

Choose a reason for hiding this comment

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

Just use a normal A record then, I don't see why you need wildcards though.

Copy link
Author

Choose a reason for hiding this comment

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

What is the difference? I thought I was applying for a normal domain anyway.

Copy link
Member

Choose a reason for hiding this comment

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

Wilcard is dsfkdsfljksf.oli-mebberson.is-cool.dev asdasdas.oli-mebberson.is-cool.dev 1223.oli-mebberson.is-cool.dev testiungs-aasdlasda.oli-mebberson.is-cool.dev to all resolve. You don't need that, you just need a normal A record:

oli-mebberson.is-cool.dev.json

Copy link
Author

@olii-dev olii-dev Jan 31, 2025

Choose a reason for hiding this comment

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

Oooh, yeah I just need a normal A record, so I will adjust that, sorry and thanks!

Edit: I can't find where I do this?

Copy link
Member

Choose a reason for hiding this comment

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

Should be the three dot menu, and hit edit file. You can then click on the name and edit it.

@DEV-DIBSTER DEV-DIBSTER added the Invalid: File File provided is mismatched or not properly formatted. label Jan 28, 2025
@open-domains
Copy link

open-domains bot commented Feb 3, 2025

This PR is stale because it has been open 3 days with no activity. Remove stale label or comment or this will be closed in 11 days.

@open-domains open-domains bot added the Awaiting Response Await response from the original contributor. label Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Response Await response from the original contributor. CLA Signed Contributor has signed the CLA Invalid: File File provided is mismatched or not properly formatted.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants