Skip to content

feat: add per-host locations seed data in Acme Org for QA#27832

Merged
Udit-takkar merged 12 commits intomainfrom
devin/1770711680-seed-per-host-locations
Feb 26, 2026
Merged

feat: add per-host locations seed data in Acme Org for QA#27832
Udit-takkar merged 12 commits intomainfrom
devin/1770711680-seed-per-host-locations

Conversation

@hariombalhara
Copy link
Member

@hariombalhara hariombalhara commented Feb 10, 2026

What does this PR do?

Adds seed data for the per-host locations feature to support manual QA of #27807. Creates a new dedicated Round Robin event type in the Acme Org → Team 1 with per-host location configuration:

  • Creates a new "Per Host Location Event" (Round Robin, 30 min) in Acme Org's Team 1
  • Enables enablePerHostLocations on the event type
  • Adds three hosts with different simple location types (no credentials required):
    • owner1-acme → Cal Video (integrations:daily)
    • member0-acme → Link (https://example.com/meet)
    • member2-acme → Organizer Phone (+1234567890)
  • Creates Host and HostLocation records for each member

The function is idempotent — it skips seeding if the event type already exists.

Updates since last revision

  • Removed credential-based app seeding: Per review feedback, removed Zoom and Google Meet mock credential creation. These caused confusion locally since the credentials weren't functional. Replaced with simple location types (Cal Video, Link, Phone) that work out of the box without any app installation or environment variables.

How should this be tested?

  1. Run yarn db-seed (or yarn dx for fresh setup)
  2. Log in as owner1-acme:owner1-acme
  3. Navigate to Event Types → Team 1 → Per Host Location Event
  4. Scroll to the Location section — the "Configure location per host" section should show different location types per host

Human Review Checklist

  • Verify location type strings (integrations:daily, link, userPhone) match the expected DefaultEventLocationTypeEnum / app location type values
  • Verify HostLocation fields (link, phoneNumber) match the Prisma schema field names
  • Confirm member0-acme and member2-acme (even-indexed) are in Team 1 per the Acme seed logic (i % 2 === 0)

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. N/A — seed-only change.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works. N/A — seed data, verified manually.

Link to Devin run: https://app.devin.ai/sessions/85638d06e725407ba486cb3c7ea3f7f5
Requested by: @hariombalhara

@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@hariombalhara hariombalhara changed the base branch from main to fix/show-all-location February 10, 2026 10:05
@github-actions github-actions bot added the ❗️ migrations contains migration files label Feb 10, 2026
@pull-request-size pull-request-size bot added size/L and removed size/M labels Feb 10, 2026
@devin-ai-integration devin-ai-integration bot changed the title feat: add per-host locations seed data for Round Robin event type feat: add per-host locations seed data in Acme Org for QA Feb 10, 2026
@hariombalhara hariombalhara marked this pull request as ready for review February 10, 2026 11:52
@graphite-app graphite-app bot added core area: core, team members only enterprise area: enterprise, audit log, organisation, SAML, SSO labels Feb 10, 2026
@graphite-app graphite-app bot requested a review from a team February 10, 2026 11:52
@hariombalhara hariombalhara requested review from Udit-takkar and removed request for a team February 10, 2026 11:52
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

devin-ai-integration[bot]

This comment was marked as resolved.

Base automatically changed from fix/show-all-location to main February 10, 2026 13:21
@github-actions
Copy link
Contributor

Devin AI is resolving merge conflicts

This PR has merge conflicts with the main branch. A Devin session has been created to automatically resolve them.

View Devin Session

Devin will:

  1. Merge the latest main into this branch
  2. Resolve any conflicts intelligently
  3. Run lint/type checks to ensure validity
  4. Push the resolved changes

If you prefer to resolve conflicts manually, you can close the Devin session and handle it yourself.

…tion seed

Co-Authored-By: hariom@cal.com <hariombalhara@gmail.com>
cubic-dev-ai[bot]

This comment was marked as resolved.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 13, 2026

Devin AI is addressing Cubic AI's review feedback

New feedback has been sent to the existing Devin session.

View Devin Session


✅ No changes pushed

@github-actions
Copy link
Contributor

E2E results are ready!

Copy link
Contributor

@Udit-takkar Udit-takkar left a comment

Choose a reason for hiding this comment

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

Image

Won't this create confusion locally and you won't be actually able to create zoom/google meet links?

Copy link
Member Author

Yeah makes sense, I was thinking only about the per-host location feature only.

let me remove these

…pes instead

Co-Authored-By: hariom@cal.com <hariombalhara@gmail.com>
@hariombalhara
Copy link
Member Author

Fixed

@github-actions
Copy link
Contributor

This PR has been marked as stale due to inactivity. If you're still working on it or need any help, please let us know or update the PR to keep it active.

@github-actions github-actions bot added the Stale label Feb 25, 2026
@Udit-takkar Udit-takkar enabled auto-merge (squash) February 26, 2026 18:39
@Udit-takkar Udit-takkar merged commit 17af50b into main Feb 26, 2026
43 checks passed
@Udit-takkar Udit-takkar deleted the devin/1770711680-seed-per-host-locations branch February 26, 2026 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only enterprise area: enterprise, audit log, organisation, SAML, SSO ❗️ migrations contains migration files ready-for-e2e size/L Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants