Skip to content

Alisha - fix for bid overview page#4073

Open
alishawalunj wants to merge 13 commits into
developmentfrom
alisha-fix-bid-overview-page
Open

Alisha - fix for bid overview page#4073
alishawalunj wants to merge 13 commits into
developmentfrom
alisha-fix-bid-overview-page

Conversation

@alishawalunj

@alishawalunj alishawalunj commented Sep 14, 2025

Copy link
Copy Markdown
Contributor

Description

D0B1AB8A-FFDA-49BE-91DC-B2F060BFE368

Related PRS (if any):

This frontend PR is related to the #1729 backend PR.

Main changes explained:

  • Integrated backend API, ensured its working on loading page and after submitting
  • Fixed issues:
    • Dark mode visibility
    • Zoom in/out images
    • Fetch user name dynamically when logged in
    • Adjusted spacing of form

How to test

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. Navigate to:
    http://localhost:5173/lbdashboard/bidOverview/681fdf349b28a081622ba779
  6. Verify that you can scroll through images and zoom in/out.
  7. Confirm that the logged-in user’s name is displayed correctly instead of USER_NAME.
  8. Submit the bid form and check that notifications appear.
  9. Verify both types of notifications:
    • Default: Bid placed successfully.
    • Highest bid: Congratulations message for placing the highest bid.
  10. Test using the following listingIds:
    • 68c5e69e7b6edc12ba7402cc
    • 68c48bec1c76310ce8e99a49
    • 68c4897c33571a094fd221f6
    • Refer to the listings collection in the DB for additional test data.
  11. Ensure the functionality works as described above.
  12. Verify that this feature also works correctly in dark mode.

Screenshots or videos of changes:

B962DF9D-54B1-48AD-827E-198C558046D5

E9DC9180-453F-49AB-8BDE-8F3F871E8420

418B11C0-6A25-45A5-93B9-E186BF019B1A

6CDAD7FC-E42D-45FE-88A5-7D6DBF5830D7

03A86CA2-1FAF-4977-A102-1798BE86D48A

@netlify

netlify Bot commented Sep 14, 2025

Copy link
Copy Markdown

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit b147ba0
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/68c643eb0c9fb20007b8832e
😎 Deploy Preview https://deploy-preview-4073--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.

@alishawalunj alishawalunj force-pushed the alisha-fix-bid-overview-page branch from b147ba0 to 92d13bc Compare April 12, 2026 21:49
@netlify

netlify Bot commented Apr 12, 2026

Copy link
Copy Markdown

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 17dfb9d
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6a361a7d1dd2a90008c21476
😎 Deploy Preview https://deploy-preview-4073--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@sonarqubecloud

Copy link
Copy Markdown

@alishawalunj alishawalunj added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Apr 13, 2026
kunchalasireesha and others added 5 commits June 19, 2026 18:26
…odNetworkApp into alisha-fix-bid-overview-page
…odNetworkApp into alisha-fix-bid-overview-page
…Constants

The bidOverviewConstants.js file is tracked in git under src/constants/lbdashboard/
(lowercase), but bidOverviewActions.js and bidOverviewReducer.js imported it from
'lbDashboard' (capital D). macOS resolves this regardless of case, but Netlify's
case-sensitive Linux build cannot resolve the module, failing the deploy.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants