Skip to content

Fix#245

Merged
S1ro1 merged 1 commit into
mainfrom
fix/no-redirect-url-in-req
Apr 15, 2025
Merged

Fix#245
S1ro1 merged 1 commit into
mainfrom
fix/no-redirect-url-in-req

Conversation

@S1ro1
Copy link
Copy Markdown
Member

@S1ro1 S1ro1 commented Apr 15, 2025

Description

Please provide a brief summary of the changes in this pull request.

Checklist

Before submitting this PR, ensure the following steps have been completed:

  • Run the slash command /verifyruns on your own server.
    • Run the cluster bot on your server:
      python discord-bot.py
    • Start training runs with the slash command /verifyruns.
    • Verify that the bot eventually responds with:
      ✅ All runs completed successfully!
      
      (It may take a few minutes for all runs to finish. In particular, the GitHub
      runs may take a little longer. The Modal run is typically quick.)
      For more information on running a cluster bot on your own server, see
      README.md.

Copilot AI review requested due to automatic review settings April 15, 2025 16:44
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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

Comments suppressed due to low confidence (1)

src/discord-cluster-manager/api/utils.py:109

  • Commenting out the 'redirect_uri' parameter may cause issues with the GitHub OAuth flow if this parameter is expected by the OAuth endpoint. Consider either permanently removing it if not needed or reintroducing it if it is necessary for the OAuth process.
# "redirect_uri": redirect_uri,

@S1ro1 S1ro1 merged commit 17a6dba into main Apr 15, 2025
3 checks 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.

3 participants