Skip to content
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

APS-1752 - Filter out rejected referrals when importing delius bookings #2771

Merged

Conversation

davidatkinsuk
Copy link
Contributor

@davidatkinsuk davidatkinsuk commented Jan 7, 2025

Imported referrals are used to backfill arrival/departure data for existing bookings, and add bookings that were created directly in delius. Given this, we are not interested in any referral from delius that was not accepted, as this would not logically result in a booking.

@davidatkinsuk davidatkinsuk changed the title APS-1752 - Filter out rejection referrals when importing delius bookings APS-1752 - Filter out rejected referrals when importing delius bookings Jan 8, 2025
@davidatkinsuk davidatkinsuk force-pushed the feature/aps-1752-filter-out-rejected-delius-imports branch from 5cc5111 to af96fac Compare January 8, 2025 09:19
@davidatkinsuk davidatkinsuk marked this pull request as ready for review January 8, 2025 09:19
@davidatkinsuk davidatkinsuk force-pushed the feature/aps-1752-filter-out-rejected-delius-imports branch from af96fac to 7b65900 Compare January 8, 2025 09:20
Imported referrals are used to backfill arrival/departure data for existing bookings, and add bookings that were created directly in delius. Given this, we are not interested in any referral from delius that was not accepted, as this would not logically result in a booking.
@davidatkinsuk davidatkinsuk force-pushed the feature/aps-1752-filter-out-rejected-delius-imports branch from 7b65900 to 19e3662 Compare January 8, 2025 09:36
@davidatkinsuk davidatkinsuk merged commit 074a616 into main Jan 8, 2025
8 checks passed
@davidatkinsuk davidatkinsuk deleted the feature/aps-1752-filter-out-rejected-delius-imports branch January 8, 2025 09:55
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.

2 participants