Skip to content
This repository has been archived by the owner on Oct 27, 2022. It is now read-only.

Add config.json file and loa slash command #4

Merged
merged 2 commits into from Jun 21, 2022

Conversation

MCUxDaredevil
Copy link
Contributor

  • Added a config file to access values dynamically instead of using constant strings
  • Added loa slash command that sets (not applies) loa for a developer and removes when they return

- Added a config file to access values dynamically instead of using constant strings
- Added loa slash command that sets (not applies) loa for a developer and removes when they return
@MrSerge01
Copy link
Collaborator

MrSerge01 commented Jun 21, 2022

Why make a config.json when we already have .env? I can add the things you added to .env and update the guide.
@MCUxDaredevil

@MCUxDaredevil
Copy link
Contributor Author

MCUxDaredevil commented Jun 21, 2022

The env can be kept for things that we don't want others to see and things that are kinda permanent or sensitive
Like the token. While the config will contain stuff that can be public like the servers ids and stuff, and things that are not quite permanent.
Also I dont think adding prefix to env vars is a good idea since we are changing it with commands
I'm not sure how directly doing process.env.PREFIX = newPrefix works but afaik the prefix will reset from the .env file every time we restart the bot

@MCUxDaredevil
Copy link
Contributor Author

MCUxDaredevil commented Jun 21, 2022

@MrSerge01 We can also use mongo to keep the config data so we don't have to modify any of the files directly
We can just change the values in the db

@MrSerge01
Copy link
Collaborator

Welp, I'm going to merge.

@MrSerge01 MrSerge01 merged commit 31b0ffe into Avdan-OS:main Jun 21, 2022
Golem642 added a commit that referenced this pull request Jun 21, 2022
Added back loa.js file
Added back config.json file (with modifications)
Credits goes to @MCUxDaredevil

Co-Authored-By: Aryan Donga <daryan6680@proton.me>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants