Skip to content

Conversation

@Lil-Pandaa
Copy link
Contributor

No description provided.

@Lil-Pandaa
Copy link
Contributor Author

Lil-Pandaa commented Dec 29, 2022

This is how it is displayed.
image

@f-o
Copy link
Collaborator

f-o commented Dec 29, 2022

Can be merged if you:

  • Fix broken path for includes directory in index.php.
  • Add the new function to the root discord.php, as per contributing guidelines.
  • Use the variable defined in the config.php for bot token, like in the other example functions.
  • Comment out the feature in login.php, and skip the manual check in the file completely. You can do a check to see if the session variable is populated on the index page, and only display the table if it is set instead.
  • Remove duplicate channels in the table. The example of checking the type should suffice.

In the future, please be wary of committing sensitive data to your repo, and especially trying to push these upstream.
While I'm able to squash the commits, to leave out the information, it will still show up in your own repo.

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