-
Notifications
You must be signed in to change notification settings - Fork 82
Added constant and enums #135
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
Conversation
|
Hey @LevelSilence Thanks for opening this PR 🚀. Mentor will review your pull request soon and till then, keep contributing and stay calm. Thanks for contributing in OpenCode'25 ✨✨! |
|
Noted the constants work, but two blockers: the workspace should be contributors//client + contributors//server (not Backend/Frontend), and the constants need to be used consistently (frontend still uses hardcoded strings). Please fix those and we’ll re-review. |
|
expecting review soon |
|
Wait we are starting to evaluate |
|
Nice progress — folder structure is fixed and backend constants are wired in. One last gap: the frontend |
|
done.... expecting review soon |
|
Hey @LevelSilence Your PR has been merged 🥳🥳 and you have earned 10 points. Thanks for contributing in OpenCode'25✨✨ |
Issue: #110
Thank you for contributing to SubSentry 🚀
Please complete this template carefully to help mentors and bots review your PR efficiently.
📌 Description
Briefly explain what you implemented in this PR.
🧩 Issue Reference
Related Issue: #110
🛠️ What Did You Implement?
Please list the major things you worked on in this PR:
(Select all that apply)
📁 Workspace Confirmation (MANDATORY)
Since all issues are Open for All, confirm the following:
contributors/<your-github-username>/client/and/orserver/files were copied from the main directory🧪 Testing Performed
Please check what applies:
📷 Screenshots / Demo (REQUIRED for most PRs)
Please attach screenshots or recordings showing:
PRs without screenshots for significant changes may be requested for updates.
✅ Final PR Checklist
Before submitting, ensure:
.envfiles committed📎 Additional Notes for Reviewers
Note: Earlier frontend and backend setup work was submitted separately and closed without my pr been merged. This PR is independent of that work and is intentionally scoped only to the constants refactor to align strictly with the current issue requirements.
Thank you for contributing to SubSentry 💙
Your effort helps make this project better for everyone 🚀