Skip to content

Conversation

@Sriamshreddy000
Copy link

Description

Screenshot 2026-01-08 at 2 39 12 AM

Related PRS (if any):

This frontend PR is related to the backend development branch.

Main changes explained:

  • Improved the Daily Equipment Log user experience by adding a clear confirmation step before submitting equipment log updates to prevent accidental changes.
  • Added visible user feedback after a successful submission so users know when an entry has been recorded.
  • Refactored the submit flow to separate user intent (submit) from execution (confirm), improving clarity and maintainability.
  • Made minor UI adjustments to ensure the confirmation banner integrates cleanly with the existing layout across screen sizes.

How to test:

  1. check into current branch
  2. do npm install and npm start to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. Navigate to Dashboard → Equipment → Daily Equipment Log
  6. Select a date and project, choose a log type (Check In or Check Out) then select one or more equipment items and click Submit, then verify the entry is submitted and also verify if a confirmation prompt appears.
  7. Click Cancel and confirm no update is made.
  8. Refresh the page and confirm data consistency.
  9. verify this new feature works in dark mode

Screenshots or videos of changes:

Submit button Submit toast message Mobile view

Note:

  • This PR focuses on UX improvements only and does not introduce backend or database changes.

@netlify
Copy link

netlify bot commented Jan 12, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit f482c7a
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69657e6641d1630008c1c260
😎 Deploy Preview https://deploy-preview-4701--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud
Copy link

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