Skip to content

Conversation

@rebelchris
Copy link
Contributor

@rebelchris rebelchris commented Dec 30, 2025

Changes

We'll just have to test it on production to be safe as this addresses current prod already.

Adds always on:

  • location check
  • level check

Events

Did you introduce any new tracking events?

Experiment

Did you introduce any new experiments?

Manual Testing

Caution

Please make sure existing components are not breaking/affected by this PR

Preview domain

https://feat-screening-additions.preview.app.daily.dev

@vercel
Copy link

vercel bot commented Dec 30, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
daily-webapp Ready Ready Preview Dec 31, 2025 10:56am
1 Skipped Deployment
Project Deployment Review Updated (UTC)
storybook Ignored Ignored Dec 31, 2025 10:56am

Copy link
Member

@idoshamun idoshamun left a comment

Choose a reason for hiding this comment

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

It's very hard to track and review. The page is very long. I suggest refactoring to smaller components

event_name: LogEvent.AnswerScreeningQuestion,
target_id: opportunityId,
extra: JSON.stringify({
question_type: 'cv',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@idoshamun I added this question_type here so we can keep track of the newly added types as well

@rebelchris rebelchris merged commit 18c5620 into main Dec 31, 2025
11 checks passed
@rebelchris rebelchris deleted the feat-screening-additions branch December 31, 2025 11:02
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.

3 participants