Commit graph

1237 commits

Author SHA1 Message Date
Oriol Roqué Paniagua
4f6728003a Merged PR 2029: Guest Journey - start date based on used link
Added verification estimated started at/date utc.
Ensured continuity for existing reports

I also noticed I missed one propagation of the end date, it's fixed here.

Note: changed the naming from 'start' to 'started' since we already had 'completed' and 'used_link'

Related work items: #17268
2024-06-13 14:53:58 +00:00
Oriol Roqué Paniagua
7914d7d8d7 Merged PR 2028: guest journey estimated completed date
This PR includes the logic to handle the completed date of guest journey

Check the schema file to understand the logic implemented. It's an estimation, and in some cases, weird stuff happens on end dates before link usage dates, which effectively breaks the computation. I added a boolean for this

Related work items: #17268
2024-06-13 13:30:22 +00:00
Oriol Roqué Paniagua
6b4490b017 Merged PR 2020: Verification request completeness for intermediate steps + schema documentation
Add more detailed logic into the verification request completeness for intermediate stages. Added schema documentation

Related work items: #17268
2024-06-13 08:14:11 +00:00
Joaquin Ossa
6ef278a271 Merged PR 2009: Added core__unified_user to reporting with the documentation
Added core__unified_user to reporting with the documentation for the dashboard for Ana and Clay

Related work items: #17146
2024-06-12 13:19:16 +00:00
Joaquin Ossa
c4f87d9403 Applied changes according to comments 2024-06-12 15:11:07 +02:00
Joaquin Ossa
33aac0be1a Added core__unified_user to reporting with the documentation 2024-06-12 15:11:07 +02:00
Oriol Roqué Paniagua
328723b9ab Merged PR 2003: Business KPIs first draft
This PR aims to provide a first draft for business KPIs, at this stage **quite minimal**.

It mostly contains a MTD and Monthly display approach. No dimensions are created.

The models created are:
- **int_dates_mtd**: mainly it reads from int_dates to retrieve those days needed for a MTD/monthly display
- **int_core__mtd_booking_metrics**: it contains the booking metrics, ending in a format of date|value|value last year|increment
--> here we could add, in the future, additional metrics from other sources (ex: guest journey, etc)
- **int_core__mtd_aggregated_metrics**: it transforms the data coming from the different mtd intermediate sources to be easily displayed in the dashboard. This is a good candidate for macros :)
- **core__mtd_aggregated_metrics**: copy-paste of the its int_core version, including a bit of model documentation

The dashboard is functional but in another PR. I can show you how it looks like at the moment you are going to check the PR

Related work items: #17214
2024-06-11 15:45:57 +00:00
Pablo Martín
366aa7d1ac Merged PR 2005: Replace Guest Details in CheckIn Cover models with details coming from User table
Fixes https://guardhog.visualstudio.com/Data/_boards/board/t/Data%20Team/Stories/?workitem=17240

Related work items: #17240
2024-06-10 15:30:26 +00:00
Pablo Martín
d18b47f597 Merged PR 2001: Add phone number to unified user
Includes the phone number field in unified user.

Also added basic tests on the PK.

Also improved upstream models traceability in the model.

Related work items: #17240
2024-06-10 14:30:56 +00:00
Pablo Martin
2ea2817627 add tests 2024-06-10 16:23:45 +02:00
Pablo Martin
723bbf1ebe add phone number 2024-06-10 16:14:07 +02:00
Pablo Martin
8b99d9babd add table aliases to track col origin 2024-06-10 15:51:40 +02:00
Joaquin Ossa
06e5112cfc Merged PR 1986: Created verification_requests model in report
Created verification_requests model in report for check-in hero report including the description to check

Related work items: #17067
2024-06-10 09:38:48 +00:00
Joaquin Ossa
1bb4229dce updated docs for core__verification_requests from Pablo's comments 2024-06-10 11:32:04 +02:00
Joaquin Ossa
5e7504499f Added description document for core__verification_requests 2024-06-06 12:00:07 +02:00
Joaquin Ossa
a5074a1545 Created verification_requests model in report 2024-06-06 10:11:34 +02:00
Pablo Martín
8ba2ece28b Merged PR 1985: Accommodation and AccommodationToUser to staging
Ingesting from Core into Staging.

Related work items: #16919
2024-06-05 15:54:48 +00:00
Pablo Martin
7aa100d115 add tests 2024-06-05 17:39:53 +02:00
Pablo Martin
85a299da96 staging accommodation to user 2024-06-05 17:38:02 +02:00
Pablo Martin
cd8fc9893d add accommodationtouser in sources 2024-06-05 17:31:19 +02:00
Pablo Martin
21f9a63db2 add test 2024-06-05 17:30:19 +02:00
Pablo Martin
26ce452a0c accommodation staging model 2024-06-05 17:24:59 +02:00
Pablo Martin
f216e7708a add accommodation to sources 2024-06-05 17:24:39 +02:00
Pablo Martin
60063b5455 propagate fix 2024-06-05 17:15:35 +02:00
Pablo Martin
c87efc7d1d low-hanging fruit refactor. Fix name 2024-06-05 16:50:14 +02:00
Pablo Martín
42a3190f50 Merged PR 1969: Integration models
This PR brings two tables from Core's `integration` schema into `staging`.
- `integration`
- `integration_type`

Related work items: #16919
2024-06-04 16:24:32 +00:00
Pablo Martin
bfd05a6410 add missing _utc suffix 2024-06-04 18:21:56 +02:00
Pablo Martin
f5ae39e443 add integration type model 2024-06-04 15:05:26 +02:00
Pablo Martin
8ec14a14c1 add integration type to sources 2024-06-04 14:58:16 +02:00
Pablo Martin
a312b1730b staging model for integration table 2024-06-04 14:51:03 +02:00
Pablo Martin
929628bc7c add integration to sources 2024-06-04 14:42:50 +02:00
Pablo Martín
76b5b7a1ea Merged PR 1954: add a few more conventions to readme
add a few more conventions to readme
2024-05-31 08:36:18 +00:00
Pablo Martin
d81c0b5e74 add a few more conventions to readme 2024-05-31 10:35:57 +02:00
Pablo Martín
ae10d30cd1 Merged PR 1952: Automated Slack Alerts
This PR:
- Improves the `run_dbt.sh` script to monitor for failures and send success/failure messages.
- Also updates the `README.md` to explain how to configure the Slack Webhook URLs.

Besides that, I've already created the right slack channels.

Related work items: #16948
2024-05-30 15:10:55 +00:00
Pablo Martin
2f661c098b updated deploy instructions in readme 2024-05-30 16:52:41 +02:00
Pablo Martin
77b4a8264c now we have both failure and success messages 2024-05-30 16:38:39 +02:00
Pablo Martin
b1aa29b228 add failure states, .env file and message sending 2024-05-30 16:12:51 +02:00
Oriol Roqué Paniagua
3610c447f5 Merged PR 1949: Fix documentation in DWH and include exposure
Related work items: #16857
2024-05-30 10:37:41 +00:00
Oriol Roqué Paniagua
dfb1fba46f Merged PR 1943: Created xero__net_fees_by_deal with guesty
Created xero__net_fees_by_deal with guesty
2024-05-30 09:49:07 +00:00
uri
f021f3cb29 Changed schema.yaml with the correct field 2024-05-30 10:03:05 +02:00
uri
e360c1c785 Setting xero__net_fees_by_deal as per deal id, not dedicated to guesty 2024-05-30 10:00:39 +02:00
uri
bf0010ceeb Adding model xero__net_fees_by_deal documentation in schema.yaml. Added Waiver Fees in xero__net_fees documentation. 2024-05-30 09:15:53 +02:00
uri
de4ab5b6e5 Created xero__net_fees_by_deal with guesty 2024-05-29 18:03:58 +02:00
Pablo Martín
822f698ce3 Merged PR 1942: add dump files to gitignore
add dump files to gitignore to help avoid fatfinger pushes of dump files.
2024-05-29 15:26:04 +00:00
Pablo Martin
fab98092c5 add dump files to gitignore 2024-05-29 17:22:48 +02:00
Pablo Martín
186d5804f8 Merged PR 1938: Add Damage Waiver to Net Fees models
Expands `xero__net_fees` with a new category dedicated to Damage Waivers.

Also fixes a small issue with the join. Should have been a full outer join from the start to properly catch months where we've only invoiced/credited some category.

Related work items: #16825
2024-05-29 11:25:14 +00:00
Pablo Martin
053024e507 add proper value to waiver_items array 2024-05-29 13:23:05 +02:00
Pablo Martin
7148220061 add net fees 2024-05-29 11:29:56 +02:00
Pablo Martin
307887126c add venv to gitignore 2024-05-29 11:27:33 +02:00
Pablo Martin
4e5424b3b6 couple of small readme details 2024-05-29 11:27:21 +02:00