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

Add Resources Section to community #192

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

CartBlanche
Copy link
Collaborator

@CartBlanche CartBlanche commented Jan 16, 2025

Added Donate button and fixed Github icon:
Screenshot 2025-01-16 at 19 08 58

@CartBlanche CartBlanche self-assigned this Jan 16, 2025
@CartBlanche
Copy link
Collaborator Author

@AristurtleDev I've used a slightly different way to you to insert the Donate button into the navbar (see screenshot above). Could you please look over the changes and let me know if I've not anything wrong and how it could be done better?

Copy link
Contributor

@AristurtleDev AristurtleDev left a comment

Choose a reason for hiding this comment

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

This looks good to me, can't say I would have done anything different. There's some small things I've noticed now looking at it that I would suggest changing in a future PR, but these are things I did in the initial setup, nothing related to your PR.

@CartBlanche
Copy link
Collaborator Author

@AristurtleDev But will these changes get around the issue you mentioned, or will this still hit the same problems you mentioned in the other PR?

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