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

checks article #1818

Merged
merged 3 commits into from
Apr 2, 2024
Merged

checks article #1818

merged 3 commits into from
Apr 2, 2024

Conversation

akiraveliara
Copy link
Member

updates the article on checks

@akiraveliara akiraveliara added docs commands For issues related to DSharpPlus.Commands labels Mar 31, 2024
docs/articles/commands/custom_context_checks.md Outdated Show resolved Hide resolved
docs/articles/commands/custom_context_checks.md Outdated Show resolved Hide resolved
docs/articles/commands/custom_context_checks.md Outdated Show resolved Hide resolved
docs/articles/commands/custom_context_checks.md Outdated Show resolved Hide resolved
Copy link
Member

@OoLunar OoLunar left a comment

Choose a reason for hiding this comment

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

I'm still not sold on it but it's better. Requesting review from Plerx and Giggity

docs/articles/commands/custom_context_checks.md Outdated Show resolved Hide resolved
@akiraveliara
Copy link
Member Author

why giggity?

@OoLunar
Copy link
Member

OoLunar commented Mar 31, 2024

why giggity?

Giggity's cool

docs/articles/commands/custom_context_checks.md Outdated Show resolved Hide resolved
docs/articles/commands/custom_context_checks.md Outdated Show resolved Hide resolved
docs/articles/commands/custom_context_checks.md Outdated Show resolved Hide resolved
@InFTord
Copy link
Member

InFTord commented Apr 2, 2024

i love how noone is dared to use docfx links, because of how often they break

@InFTord
Copy link
Member

InFTord commented Apr 2, 2024

like none of new command articles have them, which is actually great

@OoLunar
Copy link
Member

OoLunar commented Apr 2, 2024

like none of new command articles have them, which is actually great

I was actually going to ask you to fix that on my PR

@InFTord
Copy link
Member

InFTord commented Apr 2, 2024

i HATE docfx links, so, with all respect, no.

@InFTord
Copy link
Member

InFTord commented Apr 2, 2024

we can surely live without them

@OoLunar OoLunar merged commit bcd3668 into oolunar/docs/commands Apr 2, 2024
@OoLunar OoLunar deleted the checks-article branch April 2, 2024 14:32
OoLunar pushed a commit that referenced this pull request May 1, 2024
* add a few things to the checks article

* address feedback

* address PR feedback
akiraveliara added a commit that referenced this pull request May 2, 2024
* Rename `commands` to `commands_next`

- `cd $(ProjectRoot)/docs/`
- `rename "commands" "commands_next" images/commands*.png`
- `grep --files-with-matches -r "commands.*\.png" . | xargs sed -i -r -e "s/commands(.*)\.png/commands\_next\1\.png/g"`
- `mv commands commands_next`

* Rename CommandsNext article id's from `articles.commands` to `articles.commands_next`.

- `grep --files-with-matches -r "articles.commands." . | xargs sed -i "s/articles.commands./articles.commands_next./g"`

* Add deprecation notice to CommandsNext articles

* Rename Commands to CommandsNext in ToC

* First article

* Rename intro.md to introduction.md; Add custom argument converter article

* Add custom command checks article

* Add command error handling

* Consistent naming

* Fix context checks article name

* Actually call `.AddCheck`

* Autocomplete vs Choice Provider

* Formatting

* Deprecate `DSharpPlus.SlashCommands`

* checks article (#1818)

* add a few things to the checks article

* address feedback

* address PR feedback

* Deprecate CommandsNext and SlashCommands

* Remerge #1863, which mysteriously vanished

* CommandsNext isn't fully replaceable yet, for lack of help and overloads - Aki

* Document DI

* Fix hyperlinks

* CNext is unfortunately not obsoleted

* Allow the extension to use it's own obsolete classes

---------

Co-authored-by: Livia Silvercrown <61018569+akiraveliara@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
commands For issues related to DSharpPlus.Commands docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants