Skip to content

Initial commit for issue #368#406

Open
dev-rafiu wants to merge 2 commits intolmcrean:mainfrom
dev-rafiu:feature/issue-368-mobile-nav-sheet
Open

Initial commit for issue #368#406
dev-rafiu wants to merge 2 commits intolmcrean:mainfrom
dev-rafiu:feature/issue-368-mobile-nav-sheet

Conversation

@dev-rafiu
Copy link

@dev-rafiu dev-rafiu commented Oct 14, 2025

Please provide a clear and concise description of the changes made in this PR.

Screenshots / Screen Recordings

Please attach relevant screenshots or screen recordings that demonstrate UI/UX changes, Component behavior and /or Error messages (if applicable).

How to Test

Please provide step-by-step testing instructions. If any specific test data or test accounts are needed, mention them here.

  1. Pull this branch
  2. Follow setup instructions correctly as per README-developers.md
  3. Navigate to [page xyz]
  4. Verify that [specific behavior] occurs

Checklist

Before submitting this PR, the following must be confirmed:

  • Pulled the latest main branch and resolved any merge conflicts
  • Ran cd frontend && npm run build to confirm no type or build errors
  • Ran linter/formatter commands (e.g., npm run lint) and fixed warnings/errors
  • Manually tested the feature/fix on relevant browsers/devices (if UI-related)
  • Added or updated relevant documentation (if applicable)
  • Linked related issue(s) correctly in the PR description using closing keywords
  • Added Screenshots to the PR where applicable. Screenshots are mandatory for any frontend, styling, or layout changes.

Issue Links

Mention Related Issues (e.g. Fixes #123, Closes #456, as per closing keywords guidance). When updating you can also use the PR sidebar on the right-hand link via the development tab -- this will automatically close the issue when the PR gets merged

Additional Notes

(Any additional information for reviewers, like known bugs, workarounds, or future improvements.)

@dev-rafiu dev-rafiu marked this pull request as draft October 14, 2025 14:48
@lmcrean lmcrean marked this pull request as ready for review October 16, 2025 16:29
@lmcrean
Copy link
Owner

lmcrean commented Oct 16, 2025

Opened this as a test for recent deployment branching feature, can't see any sign of it working. Will revert to draft but do open when you think it's ready for review @dev-rafiu thanks

@lmcrean
Copy link
Owner

lmcrean commented Oct 16, 2025

Looking at your branch I see latest changes aren't haven't been pulled? Would you mind doing this? Curious to test if we can deploy this from the PR as a means of speeding up review process @dev-rafiu

Hope that makes sense

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