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

fix: Panel Inconsistencies and issues #952

Draft
wants to merge 12 commits into
base: development
Choose a base branch
from

Conversation

S0ly
Copy link
Member

@S0ly S0ly commented May 17, 2024

This branch focuses on resolving critical issues to prepare for production deployment. Below is the list of issues that need to be addressed:

If you discover additional issues, please inform me, and I will add them to this list.

Additionally, the codebase has some inconsistencies, such as unclosed HTML tags. Contributors are encouraged to fix these minor bugs and make any improvements where possible.

Your efforts to enhance the reliability and usability of the ctrlpanel are greatly appreciated!

Thank you for your contributions!

@S0ly S0ly added Bug / Fix Something isn't working and may need a fix High Priority Critical. Act quickly for resolution Help Wanted May need your help to resolve it labels May 17, 2024
@S0ly S0ly self-assigned this May 17, 2024
@S0ly S0ly marked this pull request as draft May 17, 2024 12:45
@S0ly S0ly added this to the V0.10 milestone May 18, 2024
@S0ly S0ly changed the title Fix Panel Inconsistencies and issues fix: Panel Inconsistencies and issues Jun 4, 2024
S0ly and others added 2 commits June 17, 2024 10:09
Caveats: .env MUST contain DISCORD_CLIENT_ID and DISCORD_CLIENT_SECRET, otherwise insertion will still fail (NEEDS TO BE DOCUMENTED AS BREAKING!)
Copy link
Collaborator

@AGuyNamedJens AGuyNamedJens left a comment

Choose a reason for hiding this comment

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

.env file for every ctrlpanel user must now contain the variables DISCORD_CLIENT_ID and DISCORD_CLIENT_SECRET, otherwise the method will not work on their computers. Add this to the documented breaking changes for the new version?

@jameskitt616
Copy link

move the file transferusers.php somewhere else then the root of the project..

attempted to fix this issue with #976

@S0ly
Copy link
Member Author

S0ly commented Sep 6, 2024

guys anyone want to fix the remaining issues ?

@jameskitt616
Copy link

I opened a PR (#1002) for #989

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment