Update bot token docs by Yudi · Pull Request #1371 · Tyrrrz/DiscordChatExporter (original) (raw)

Closes #1370

Also added step "5. Click Yes, do it! and authenticate to confirm" to GUI and CLI as users might find it unusual for a popup to request their password/2fa code.

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the bot token documentation and related UI text to reflect the new process for resetting a bot token. Key changes include:

• Updating the CLI guide to instruct users to "Reset Token" instead of copying the token.
• Adding a step for user authentication confirmation.
• Adjusting documentation links and images in the troubleshooting and token guides.

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

Files not reviewed (1)

• DiscordChatExporter.Gui/Views/Components/DashboardView.axaml: Language not supported

Thanks for the PR.

Hmm, maybe we shouldn't direct users to reset their token by default, as it may break their other integrations, if they have any.

Can we preface the list of steps that explains that you should've gotten the token when you first created the bot? Otherwise, direct them through the steps to reset it if they forgot.

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the bot token documentation to clarify the token reset process in both the CLI guide and the documentation files.

• In the CLI command, the instructions now specify that users should reset the token rather than simply copy it, and an extra step for authenticating the reset is added.
• The troubleshooting and token documentation files have been revised with updated links and additional instructions to help users securely obtain and manage their tokens.

Reviewed Changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated 1 comment.

Files not reviewed (2)

• DiscordChatExporter.Gui/Views/Components/DashboardView.axaml: Language not supported
• DiscordChatExporter.Gui/Views/MainView.axaml: Language not supported

Co-authored-by: Copilot 175728472+Copilot@users.noreply.github.com

Well noticed.
New text implies user should've saved the token and makes it explicit the old token will stop working.

@Tyrrrz do you think this order makes sense and that the layout looks good?
"If the token was not saved…" isn't an important notice, so an * shouldn't precede it(?)

New changes:

• Add complete steps to docs
  • App creation, bot invitation, etc
• Add "If the token was not saved during bot creation, a new one can be generated:"
• Add "Integrations using the old token will stop working until updated"

GUI:

• Bump <Window> Height to 665 to fit content
  • Number was chosen arbitrarily

Previous 625 height

I like the new layout. I would rephrase the first line a bit to highlight a bit more that the user should already have the token:

Token is generated when you create the bot. If you've lost it, follow the steps below to generate a new one:

If the line gets too wide, feel free to insert a line break after the period.

As for the *, I think the way it is in the last screenshot is perfect.

Thanks! Just some small nit picks and we're done

Yudi and others added 2 commits

May 4, 2025 13:44

Co-authored-by: Oleksii Holub 1935960+Tyrrrz@users.noreply.github.com

Ah, didn't realize that affected rendering

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})