Skip to content

Conversation

@LevelSilence
Copy link
Contributor

Issue: #110

⚠️ This PR must be linked to a valid OpenCode issue number.
PRs without an issue number may not be reviewed.


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.

  • What problem does it solve?
  • What part of the project does it affect?

🧩 Issue Reference

Related Issue: #110

Note: Do NOT use words like Closes, Fixes, or Resolves.
Issues are Open for All and should not be auto-closed.


🛠️ What Did You Implement?

Please list the major things you worked on in this PR:

  • Feature implementation
  • UI changes
  • API / backend logic
  • Refactor / cleanup
  • Documentation update
  • Demo / setup task

(Select all that apply)


📁 Workspace Confirmation (MANDATORY)

Since all issues are Open for All, confirm the following:

  • All changes are inside
    contributors/<your-github-username>/
  • Base client/ and/or server/ files were copied from the main directory
  • No files outside my personal workspace were modified

🧪 Testing Performed

Please check what applies:

  • Frontend runs locally without errors
  • Backend server runs locally without errors
  • APIs tested with valid inputs
  • APIs tested with invalid / edge-case inputs
  • No console errors or warnings

📷 Screenshots / Demo (REQUIRED for most PRs)

Please attach screenshots or recordings showing:

  • Folder structure
  • Major code changes
  • UI changes (if applicable)
  • Server / client running (for setup or demo issues)

PRs without screenshots for significant changes may be requested for updates.


✅ Final PR Checklist

Before submitting, ensure:

  • This PR addresses only one issue
  • Issue number is mentioned at the top
  • Code is readable and well-structured
  • No unnecessary files are included
  • No sensitive data or .env files committed
  • Commit messages follow project conventions
  • PR title is clear and descriptive

📎 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 🚀

@OpenGitBot
Copy link

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 ✨✨!

@Aryan-coder06
Copy link
Collaborator

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.

@LevelSilence
Copy link
Contributor Author

expecting review soon

@Aryan-coder06
Copy link
Collaborator

Wait we are starting to evaluate

@Aryan-coder06
Copy link
Collaborator

Nice progress — folder structure is fixed and backend constants are wired in. One last gap: the frontend
constants file is still unused. Please either use those constants in the client (even a simple usage
example) or remove the client constants so it’s not dead code. After that, we can merge

@LevelSilence
Copy link
Contributor Author

LevelSilence commented Jan 3, 2026

done.... expecting review soon

@Aryan-coder06 Aryan-coder06 merged commit 77210ec into opencodeiiita:main Jan 5, 2026
@OpenGitBot
Copy link

Hey @LevelSilence

Your PR has been merged 🥳🥳 and you have earned 10 points.

Thanks for contributing in OpenCode'25✨✨

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.

3 participants