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

Body, Url Added #98

Merged
merged 3 commits into from
Jun 25, 2024
Merged

Body, Url Added #98

merged 3 commits into from
Jun 25, 2024

Conversation

end3r-man
Copy link
Contributor

Add the doc about the body, and the url

@Log1x
Copy link
Member

Log1x commented Jun 25, 2024

Thank you! ->body() is already at the bottom of Basic Usage though 👀 – perhaps it should be moved? I was thinking of expanding the section up there that might make it more notice-able though with the addition of the methods from #94

@Log1x
Copy link
Member

Log1x commented Jun 25, 2024

Actually let's keep the second mention of ->body(), but move it below ->content() on the page and table of contents. Also if you wouldn't mind capitalizing URL in the table of contents, then this is good to go. 😊

@end3r-man
Copy link
Contributor Author

end3r-man commented Jun 25, 2024

->body() as u mention right now see the body in the docs so in my point its nice to have in the list easy to notice

@end3r-man
Copy link
Contributor Author

Actually let's keep the second mention of ->body(), but move it below ->content() on the page and table of contents. Also if you wouldn't mind capitalizing URL in the table of contents, then this is good to go. 😊

no problem ill update it

@end3r-man
Copy link
Contributor Author

moved the body under the content and caps the Url in the list

@Log1x Log1x merged commit 297ec16 into laracord:main Jun 25, 2024
1 check passed
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