Skip to content

sandbox to main#67

Closed
ObeeJ wants to merge 380 commits intomainfrom
sandbox
Closed

sandbox to main#67
ObeeJ wants to merge 380 commits intomainfrom
sandbox

Conversation

@ObeeJ
Copy link
Copy Markdown
Contributor

@ObeeJ ObeeJ commented Feb 27, 2026

No description provided.

ObeeJ and others added 30 commits January 29, 2026 10:23
…Hero` and `Navbar`, and temporarily comment out the `TestimonialsSection` in `Home.tsx`.
refactor: replace external images with local assets, update Logo, and add social links
merge(sandbox): integrate latest sandbox into drEnglish with conflict resolution favoring local
feat(property): implement tenant viewing request modal and booking fl…
Refactor scout viewing workflow, implement media uploader, and add vi…
feat: replace booking form with inspection fee confirmation modal
feat(tenant): implement inspection payment flow with responsive UI
feat: implement payment feedback component and integrate with viewing…
…cing commitment UX by removing escape routes, and adding landlord ownership proof request
Refactor booking flow: Implementing post-payment schedule page, enfor…
Fix: added a validation logic to applications
Starr365 and others added 30 commits March 5, 2026 22:03
…ax error

- Add uploadTimeout (120s) to env config separate from apiTimeout (30s)
- Wire uploadTimeout into document, image, and video upload mutations
- Remove stray closing brace in RoleOnboarding.tsx that caused build error
The Axios instance defaults to Content-Type: application/json which
overrides the browser's automatic multipart/form-data boundary on
FormData requests, causing the upload service to receive no file.
Setting Content-Type to undefined lets the browser set the correct
header including the required boundary string.
PhotosUploader calls postData directly instead of using the upload
hook, so the earlier hook fix did not cover this path. Same root cause:
Axios default Content-Type: application/json overrides FormData multipart
boundary, causing the backend to receive no file.
… bleed

A multi-role user who owns a property was seeing "Update Listing" while
browsing as TENANT because isOwner only checked user ID, not active role.
Now requires both landlordId match and currentRole === LANDLORD.
/tenant/dashboard and /scout/dashboard don't exist; correct paths are
/tenant and /scout (index routes).
…n layout

- AuthLayout: auto-advancing 2-image carousel (5s interval) with fade
  transition, dot indicators; second slide uses real-estate-1.jpg
- Login: move "Forgot password?" to right-aligned below password field,
  remove overlapping session note text
Show only backend-allowed next statuses based on current application state:
SUBMITTED → UNDER_REVIEW, UNDER_REVIEW → SHORTLISTED/REJECTED,
SHORTLISTED → ACCEPTED/REJECTED. Prevents 400 errors from invalid transitions.

Also fix fetchPriority casing in AdminLogin.
When the KYC API returns an empty government_name, auto-start in manual
entry mode and remove the duplicate updateName call that was sending
empty first/last name fields to the backend.
Tenant: apply page hard-blocked; applications/tenancy/viewings/
agreements/payments/maintenance show pending overlay.
Landlord: dashboard/keys/viewing-windows/applications/agreements/
finance show overlay; properties list shows restricted badge;
create/edit property hard-blocked.
Scout: tasks/keys/incidents/verification/availability show overlay.
Admin/Agent bypass all KYC checks as before.
…vailable

Prevents user from navigating to a dead-end state where Use Government
Name is disabled and there is nothing to go back to.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Remove route-level KycGuard wrappers from all landlord pages.
Gate at the two critical actions instead:
- CreateProperty: KYC check fires on final step submit, holds form
  data in state and resumes after verification
- ApplicationDetailsModalContent + MyApplicationsTable: KYC check
  fires only when landlord selects ACCEPTED, then opens decision modal
  on success
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.

4 participants