Skip to content

docs: narrow Guestline imports to three reports and add synthetic exports - #116

Open
Alexcchip wants to merge 3 commits into
mainfrom
docs/83-guestline-exports
Open

Alexcchip wants to merge 3 commits into
mainfrom
docs/83-guestline-exports

Conversation

@Alexcchip

@Alexcchip Alexcchip commented Oct 25, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Records what the first Guestline exports contain, narrows the import to
three reports, and decides how often each is uploaded. Daily Rate Analysis
and Rate Override were dropped after review. Adds synthetic exports for
tests. Refs #83.

Type of change

  • docs — documentation only
  • test — adding or updating tests

Changes

  • New decision 0032 supersedes 0031: Daily New Reservations, Cancellations
    and Actual & Forecast only.
  • All three are uploaded daily: a one-off 2-year backfill, then the last 7
    days of each report every day. Actual & Forecast stops at today, since
    future nights come from the booking rebuild.
  • docs/guestline-data.md: coverage, columns removed, answers to 0031's open
    questions, and the rebuild check (matches Total on every night since
    Jan 2025).
  • Daily New Reservations has no cancelled bookings, so the rebuild needs
    Cancellations too. That report must be filtered by cancellation date.
  • Real exports move to gitignored backend/hidden_data/. backend/data/ now
    holds synthetic CSVs from data/generate.py.

Database migrations

  • No schema changes

Checklist

  • Commits follow Conventional Commits
  • Lint passes
  • Updated docs

How to test

  1. cd backend && python3 data/generate.py prints counts and exits cleanly.
    The script asserts the bookings add up to Total on every night.
  2. git status shows no files under backend/hidden_data/.

Notes for reviewers

  • 0031 is superseded rather than edited, per docs/decisions/README.md.
  • Still open: DFS, amended bookings, and why Accomm sits between nett and
    gross.

@Alexcchip Alexcchip linked an issue Oct 25, 2026 that may be closed by this pull request
4 tasks done
Record the split in 0032: one 2-year backfill, then the last 7 days of each
report every day. Actual & Forecast stops at today, since future nights are
covered by the booking rebuild. Add 0032 to the decision index.

Refs #83
@Alexcchip

Copy link
Copy Markdown
Collaborator Author

Dropping Daily Rate Analysis and Rate Override affects #85 and #86 (both can close as not planned) and #89 (drop them from its blockers).

@Alexcchip Alexcchip self-assigned this Oct 25, 2026
@Alexcchip

Alexcchip commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator Author

The three still-open items (DFS, amended bookings, and why Accomm sits between nett and gross) are much easier to settle by checking the Rezlynx portal than by deriving them from the exports. Each is a few minutes in the report options, the reservation logs, or one booking's bill. Parking them until we have the login.

@selvarajuv @emmagershman is it possible to get login access? if not can direct you guys towards where we would probably figure this out. Also none of these are blocking, but the ammended bookings question helps with tracking changes to stay periods (someone changes an arrival/departure date after booking)

@Alexcchip
Alexcchip marked this pull request as ready for review October 26, 2026 01:33
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.

Collect two years of Guestline exports from the hotel

1 participant