-
Notifications
You must be signed in to change notification settings - Fork 11.6k
fix(companion): iOS event type detail fixes #26845
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Fix #1: Weekly schedule now shows multiple time slots per day in AvailabilityTab - Fix #2: Allow numeric input fields to be empty with proper validation - Fix #3: Add native iOS date picker for date range in Limits tab - Fix #4: Change 'Email verification' to 'Booker email verification' in Advanced tab - Fix #5: Fix Requires confirmation comparison logic in buildPartialUpdatePayload - Fix #6: Forward parameters toggle already working, verified wiring - Fix #7: Add toggles for redirect booking URL and interface language in Advanced tab - Fix #8: Increase scroll padding at bottom from 200 to 350 for better keyboard access
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 issues found across 8 files
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="companion/components/LoginScreen.tsx">
<violation number="1" location="companion/components/LoginScreen.tsx:73">
P3: Frontend UI strings must be wrapped in `t()` for localization; the reintroduced sign‑up prompt is hardcoded English text.</violation>
</file>
<file name="companion/components/event-type-detail/tabs/LimitsTabDatePicker.ios.tsx">
<violation number="1" location="companion/components/event-type-detail/tabs/LimitsTabDatePicker.ios.tsx:4">
P3: Date pickers don’t support placeholders but this component’s props expose a `placeholder` that is never used, creating a misleading API surface and dead prop.
(Based on your team's feedback about DatePicker placeholder props.) [FEEDBACK_USED]</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
Devin AI is addressing Cubic AI's review feedbackA Devin session has been created to address the issues identified by Cubic AI. |
What does this PR do?
This PR addresses 8 issues found during iOS app testing in the companion folder event type detail screen:
find()tofilter())|| "0")buildPartialUpdatePayload.tsfor proper change detectionUpdates since last revision
Mandatory Tasks (DO NOT REMOVE)
How should this be tested?
Checklist
Human Review Checklist
requiresConfirmationcomparison logic change inbuildPartialUpdatePayload.ts(lines 643-651) doesn't break existing functionalityLink to Devin run: https://app.devin.ai/sessions/9dea3de9fe1c4fb5a2188c3ef863ede1
Requested by: Dhairyashil Shinde (@dhairyashiil)