Skip to content

Fix error in botstrap when attempting to upgrade to community server#3528

Open
wookie184 wants to merge 2 commits into
mainfrom
wookie/fix-botstrap-community-patch
Open

Fix error in botstrap when attempting to upgrade to community server#3528
wookie184 wants to merge 2 commits into
mainfrom
wookie/fix-botstrap-community-patch

Conversation

@wookie184

@wookie184 wookie184 commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Previously it was sending the entire guild info payload in the patch. Now it only sends the fields we want to change.

Also send a helpful error message if the bot doesn't have admin permisssions on the server.

Testing:

  • Confirmed the friendly message is sent without admininstrator permission.
  • Confirmed bootstrap now works when reinvited with administrator permission.

Comment thread botstrap.py
)
except HTTPStatusError as e:
if e.response.status_code == 403:
log.error("Unable to upgrade to community guild as he bot does not have Administrator permissions")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
log.error("Unable to upgrade to community guild as he bot does not have Administrator permissions")
log.error("Unable to upgrade to community guild as the bot does not have Administrator permissions.")

Comment thread botstrap.py
if e.response.status_code == 403:
log.error("Unable to upgrade to community guild as he bot does not have Administrator permissions")
log.info(
"Please re-invite with the following URL and rerun this script: "

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
"Please re-invite with the following URL and rerun this script: "
"Please re-invite with the following URL and run this script again: "

This wording might be a bit more understandable to non-native English speakers.

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