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

Modify readme #36

Merged
merged 11 commits into from
Aug 1, 2024
Merged

Modify readme #36

merged 11 commits into from
Aug 1, 2024

Conversation

jspmic
Copy link
Collaborator

@jspmic jspmic commented Jul 27, 2024

  • [ x ] Project's Image
  • [ x ] Installation Procedure
  • [ x ] Setup Procedure
  • [ ] Bot's Usage
  • [ x ] Conclusion and Key Achievements

jspmic and others added 3 commits July 27, 2024 19:35
- Added a conclusion
- Added the Installation procedure
- Added the Setup procedure
README.md Outdated Show resolved Hide resolved

3. **Set Up Your Environment Variables**:
- Create a .env file in the root of your project and add your Discord API Token: `DISCORD_BOT_KEY="your-discord-token-here"`

Copy link
Owner

Choose a reason for hiding this comment

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

We should have a step for updating the emojis used in config.py. They should change the IDs to their own emojis' IDs, after they add the emojis under bot/assets/icons to a server or their bot application. Otherwise the bot won't work (unless its invited into our server where the emojis already exist)

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated

3. add or set `DISCORD_BOT_KEY=[your_bot_key]` with bot's key.
For more information, Consult The [Discord Developer Portal](https://discord.com/developers/docs/intro) Documentation.
Copy link
Owner

Choose a reason for hiding this comment

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

We also have a dev guide for setting up the bot in discord's dev portal. We should either link to that or integrate that text into the README. IMO integrating it into the readme would make the most sense.

README.md Outdated Show resolved Hide resolved
@gustavwilliam gustavwilliam merged commit 6ace6fd into main Aug 1, 2024
1 check 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