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 README #31

Closed
wants to merge 3 commits into from
Closed

Update README #31

wants to merge 3 commits into from

Conversation

JakobTopholt
Copy link
Contributor

First and final commit, for some reason it loos like I changed more than I did.

F-klub is a ferrari club, F-klubben is the one we are.

First and final commit, for some reason it loos like I changed more than I did.

F-klub is a ferrari club, F-klubben is the one we are.
@JakobTopholt JakobTopholt linked an issue Jun 9, 2024 that may be closed by this pull request
@JakobTopholt JakobTopholt requested a review from falkecarlsen June 9, 2024 10:17
@krestenlaust
Copy link
Member

The formatting looks great, but I don't agree that F-club is the Ferrari club. We commonly refer to ourselves as the F-club in english. For example, the emails we send to our users in stregsystem, and our domain name too.

@JakobTopholt
Copy link
Contributor Author

http://f-klub.dk/

@krestenlaust
Copy link
Member

I know the domain already, you think I'm a rookie😏 doesn't change the fact that we commonly refer to F-Klubben as "the F-club" in english

@JakobTopholt JakobTopholt requested review from jones-g and removed request for falkecarlsen June 17, 2024 07:40
@JakobTopholt JakobTopholt requested a review from mads256h June 17, 2024 09:24
README.md Show resolved Hide resolved
## Adding new songs


=======
Copy link
Member

Choose a reason for hiding this comment

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

billede
I'm not sure how this is supposed to look

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ooops, Imma fix it.

@krestenlaust
Copy link
Member

@JakobTopholt JakobTopholt deleted the 30-fix-readme branch July 16, 2024 08:04
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.

Fix README
2 participants