Skip to content

Fix: Fixed converting discord user id to string to find it and storing github user id as string - #102

Merged
sach-12 merged 10 commits into
pesu-dev:devfrom
Yashneel7:Neelbox/contribute-org-member
Oct 16, 2026
Merged

sach-12 merged 10 commits into
pesu-dev:devfrom
Yashneel7:Neelbox/contribute-org-member

Conversation

@Yashneel7

@Yashneel7 Yashneel7 commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

📌 Description


🧱 Type of Change

  • 🐛 Bug fix – Non-breaking fix for a functional/logic error
  • ✨ New feature – Adds functionality without breaking existing commands or APIs
  • ⚠️ Breaking change – Backward-incompatible change (commands, bot behavior, etc.)
  • 📝 Documentation update – README, comments, help text, etc.
  • 🧪 Test suite change – Adds/updates unit, integration, or manual tests
  • ⚙️ CI/CD pipeline update – GitHub Actions, Docker, Git hooks, etc.
  • 🧹 Refactor – Code cleanup, improvements, or style changes
  • 🐢 Performance improvement – Faster command responses or reduced resource use
  • 🕵️ Logging/debugging – Improved diagnostics, logs, or debug output
  • 🔧 Tooling – Scripts, benchmarks, or local dev improvements
  • 🔒 Security fix – Permissions, validation, or Discord API-related vulnerabilities
  • 🧰 Dependency update – Library or package updates

🧪 How Has This Been Tested?

  • Unit Tests (tests/unit/)
  • Integration Tests (tests/integration/)
  • Manual testing in Discord
  • CI

Test Environment:

  • OS: (e.g., Linux)
  • Python: (e.g., 3.13)
  • Discord.py version: (e.g., 2.7.x)
  • Docker build tested

✅ Checklist

  • My code follows repo CONTRIBUTING.md guidelines
  • Self-review completed
  • Added/updated comments and docstrings
  • Updated relevant docs (README, help text, etc.)
  • No new warnings or errors introduced
  • Added/updated tests
  • All tests pass locally
  • Ran linting and formatting
  • Docker image builds and runs
  • Changes are backwards compatible (if applicable)
  • Feature flags or .env vars updated (if applicable)
  • Tested in multiple environments (if applicable)

🛠️ Affected Bot Areas

  • Commands (e.g., /ping, /help)
  • Event handling (on_message, on_ready, etc.)
  • Permissions/roles
  • Database/Storage
  • Discord API/Integrations
  • Bot startup/shutdown
  • CI / Git hooks
  • Dockerfile / build
  • Dependencies (pyproject.toml)
  • Tooling/scripts
  • Other (specify in Additional Notes)

📸 Screenshots / Demos (if applicable)


🧠 Additional Notes

@Yashneel7
Yashneel7 requested review from a team and sach-12 as code owners October 13, 2026 18:31
@Yashneel7
Yashneel7 enabled auto-merge October 13, 2026 18:56
@sach-12
sach-12 disabled auto-merge October 16, 2026 00:08
@sach-12
sach-12 added this pull request to the merge queue Oct 16, 2026
Merged via the queue into pesu-dev:dev with commit bd37305 Oct 16, 2026
6 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.

2 participants