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

docs: add custom styling & custom logo #531

Merged
merged 20 commits into from Jan 31, 2022
Merged

Conversation

thim81
Copy link

@thim81 thim81 commented Jan 25, 2022

Description

Changes proposed in this pull request:

  • add documentation on how to apply custom CSS styling
  • add documentation on how to show a custom logo

Related issue(s)
#530

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide and the instructions about a basic recommended setup useful for opening a pull request.

Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

@magicmatatjahu
Copy link
Member

@thim81 Hi! I will review tomorrow, don't worry :)

README.md Outdated Show resolved Hide resolved
thim81 and others added 2 commits January 26, 2022 09:00
Co-authored-by: Samriddhi <Agrawalsamriddhi83@gmail.com>
@thim81
Copy link
Author

thim81 commented Jan 26, 2022

If possible do a "Squash and merge" since the spelling & indenting corrections are too much :-)

Copy link
Member

@magicmatatjahu magicmatatjahu left a comment

Choose a reason for hiding this comment

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

@thim81 Thanks for awesome contribution! However I have 7 suggestions how we should improve new docs :) Please refer to them.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@magicmatatjahu
Copy link
Member

@thim81

If possible do a "Squash and merge" since the spelling & indenting corrections are too much :-)

Don't worry, we always squash before merge, we have enabled it on the Github side :)

thim81 and others added 6 commits January 26, 2022 13:02
Co-authored-by: Maciej Urbańczyk <urbanczyk.maciej.95@gmail.com>
Co-authored-by: Maciej Urbańczyk <urbanczyk.maciej.95@gmail.com>
Co-authored-by: Maciej Urbańczyk <urbanczyk.maciej.95@gmail.com>
Co-authored-by: Maciej Urbańczyk <urbanczyk.maciej.95@gmail.com>
Co-authored-by: Maciej Urbańczyk <urbanczyk.maciej.95@gmail.com>
Co-authored-by: Maciej Urbańczyk <urbanczyk.maciej.95@gmail.com>
@sonarcloud
Copy link

sonarcloud bot commented Jan 26, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Member

@magicmatatjahu magicmatatjahu left a comment

Choose a reason for hiding this comment

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

@thim81 Hi! Please ad in some places missed "" chars because I see that we have broken md file. Github suggestion system has problem with ""...

@thim81
Copy link
Author

thim81 commented Jan 26, 2022

@magicmatatjahu The missing quotes are back and the page should look OK again.

Copy link
Member

@magicmatatjahu magicmatatjahu left a comment

Choose a reason for hiding this comment

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

Ok, last suggestion and for me everything looks good! @alequetzalli Do you wanna also make review?

README.md Outdated Show resolved Hide resolved
Co-authored-by: Maciej Urbańczyk <urbanczyk.maciej.95@gmail.com>
Copy link
Member

@magicmatatjahu magicmatatjahu left a comment

Choose a reason for hiding this comment

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

LGTM! We should wait for response from @alequetzalli :)

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link
Member

@alequetzalli alequetzalli left a comment

Choose a reason for hiding this comment

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

Left my review! ✨❤️ Just a few suggestions.

@magicmatatjahu @thim81

thim81 and others added 2 commits January 27, 2022 09:25
Co-authored-by: Alejandra Quetzalli  <alejandra.quetzalli@postman.com>
@magicmatatjahu
Copy link
Member

@alequetzalli Thanks for review! :)

README.md Outdated Show resolved Hide resolved
@sonarcloud
Copy link

sonarcloud bot commented Jan 27, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Member

@alequetzalli alequetzalli left a comment

Choose a reason for hiding this comment

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

🚀 LGTM

@thim81
Copy link
Author

thim81 commented Jan 28, 2022

@magicmatatjahu According to Github there is a still a change request open? But from what I can see all the comments have been resolved? Can you tell me if you were missing something?

@magicmatatjahu
Copy link
Member

@thim81 Hi! I felt bad on Thursday and Friday and didn't work. Sorry for the delay. Everything is ok. Thanks for your contribution! ❤️

Copy link
Member

@magicmatatjahu magicmatatjahu left a comment

Choose a reason for hiding this comment

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

LGTM!

@magicmatatjahu magicmatatjahu merged commit a59272a into asyncapi:next Jan 31, 2022
@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 1.0.0-next.33 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants