Skip to content

feat: waitlist lottery - listing detail page states update#5575

Merged
millord merged 3 commits into
mainfrom
5507/waitlist-lottery-listing-detail-page-states-update-core
Nov 24, 2025
Merged

feat: waitlist lottery - listing detail page states update#5575
millord merged 3 commits into
mainfrom
5507/waitlist-lottery-listing-detail-page-states-update-core

Conversation

@millord

@millord millord commented Nov 14, 2025

Copy link
Copy Markdown
Collaborator

This PR addresses #5507

  • Addresses the issue in full
  • Addresses only certain aspects of the issue

Description

The purpose of this change was display also -> " Waitlist is open - submit an application for an open slot on the waitlist" when the listing type is waitlist lottery .

The change was in one place but includes both SHOW_NEW_SEEDS_DESIGNS=TRUE/TRUE

Please include a summary of the change and which issue(s) is addressed.

How Can This Be Tested/Reviewed?

  1. Create a waitlistLottery listing in the partners site
  2. Navigate to the public site (Core) and find that listing
  3. Open the listing
  4. In the listing detail page should say Waitlist is open
image

2-With SHOW_NEW_SEEDS_DESIGNS=TRUE

image

Provide instructions so we can review, including any needed configuration, and the test cases that need to be QAd.

Author Checklist:

  • Added QA notes to the issue with applicable URLs
  • Reviewed in a desktop view
  • Reviewed in a mobile view
  • Reviewed considering accessibility
  • Added tests covering the changes
  • Made corresponding changes to the documentation
  • Ran yarn generate:client and/or created a migration when required

Review Process:

  • Read and understand the issue
  • Ensure the author has added QA notes
  • Review the code itself from a style point of view
  • Pull the changes down locally and test that the acceptance criteria is met
  • Either (1) explicitly ask a clarifying question, (2) request changes, or (3) approve the PR, even if there are very small remaining changes, if you don't need to re-review after the updates

@millord millord added the 1 review needed Requires 1 more review before ready to merge label Nov 14, 2025
@millord millord changed the title added waitlistlottery status to message check for list view feat: waitlist Lottery - Listing Detail Page States Update Nov 14, 2025
@netlify

netlify Bot commented Nov 14, 2025

Copy link
Copy Markdown

Deploy Preview for partners-bloom-dev ready!

Name Link
🔨 Latest commit 37246d5
🔍 Latest deploy log https://app.netlify.com/projects/partners-bloom-dev/deploys/6924ae7fd2dbbd00082e583e
😎 Deploy Preview https://deploy-preview-5575--partners-bloom-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.

@netlify

netlify Bot commented Nov 14, 2025

Copy link
Copy Markdown

Deploy Preview for bloom-exygy-dev ready!

Name Link
🔨 Latest commit 37246d5
🔍 Latest deploy log https://app.netlify.com/projects/bloom-exygy-dev/deploys/6924ae7ff49862000811205a
😎 Deploy Preview https://deploy-preview-5575--bloom-exygy-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.

@netlify

netlify Bot commented Nov 14, 2025

Copy link
Copy Markdown

Deploy Preview for bloom-angelopolis ready!

Name Link
🔨 Latest commit 37246d5
🔍 Latest deploy log https://app.netlify.com/projects/bloom-angelopolis/deploys/6924ae7f21a1c70008090490
😎 Deploy Preview https://deploy-preview-5575--bloom-angelopolis.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.

@netlify

netlify Bot commented Nov 14, 2025

Copy link
Copy Markdown

Deploy Preview for bloom-public-seeds ready!

Name Link
🔨 Latest commit 37246d5
🔍 Latest deploy log https://app.netlify.com/projects/bloom-public-seeds/deploys/6924ae7ff74e7600086cfee2
😎 Deploy Preview https://deploy-preview-5575--bloom-public-seeds.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.

@netlify

netlify Bot commented Nov 14, 2025

Copy link
Copy Markdown

Deploy Preview for bloom-flagly ready!

Name Link
🔨 Latest commit 37246d5
🔍 Latest deploy log https://app.netlify.com/projects/bloom-flagly/deploys/6924ae7fff65c10008fa478d
😎 Deploy Preview https://deploy-preview-5575--bloom-flagly.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.

@emilyjablonski

Copy link
Copy Markdown
Collaborator

@ludtkemorgan Are we not also making the changes in Seeds pages for whenever Doorway moves to Seeds?

@netlify

netlify Bot commented Nov 14, 2025

Copy link
Copy Markdown

Deploy Preview for bloom-lakeview ready!

Name Link
🔨 Latest commit 37246d5
🔍 Latest deploy log https://app.netlify.com/projects/bloom-lakeview/deploys/6924ae7ffaee2600084488ec
😎 Deploy Preview https://deploy-preview-5575--bloom-lakeview.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.

@mcgarrye mcgarrye changed the title feat: waitlist Lottery - Listing Detail Page States Update feat: waitlist lottery - listing detail page states update Nov 18, 2025
@mcgarrye mcgarrye added the needs changes The author must make changes and then re-request review before merging label Nov 18, 2025

@ludtkemorgan ludtkemorgan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@ludtkemorgan ludtkemorgan added ready to merge Should be applied when a PR has been reviewed and approved and removed needs changes The author must make changes and then re-request review before merging 1 review needed Requires 1 more review before ready to merge labels Nov 24, 2025
@millord millord merged commit a617f10 into main Nov 24, 2025
34 checks passed
@millord millord deleted the 5507/waitlist-lottery-listing-detail-page-states-update-core branch November 24, 2025 21:56
ludtkemorgan pushed a commit to metrotranscom/doorway that referenced this pull request Nov 24, 2025
…sing#5575)

show  waitlist open when listing type is waitlist lottery
ludtkemorgan pushed a commit to metrotranscom/doorway that referenced this pull request Nov 24, 2025
…sing#5575)

show  waitlist open when listing type is waitlist lottery
ludtkemorgan pushed a commit to metrotranscom/doorway that referenced this pull request Dec 8, 2025
…sing#5575)

show  waitlist open when listing type is waitlist lottery
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Should be applied when a PR has been reviewed and approved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants