Skip to content

[IMP] real_estate: update appointment type demo data#2022

Closed
chga-odoo wants to merge 1 commit into
odoo:19.0from
odoo-dev:saas-19.2-real-estate-appointment-chga
Closed

[IMP] real_estate: update appointment type demo data#2022
chga-odoo wants to merge 1 commit into
odoo:19.0from
odoo-dev:saas-19.2-real-estate-appointment-chga

Conversation

@chga-odoo
Copy link
Copy Markdown
Contributor

  • Add appointment_type demo with slots
  • Set appointment defaults in server action
  • Link calendar events via x_property_id
  • Update event timings and timezone handling

task: 6050591

@robodoo
Copy link
Copy Markdown
Collaborator

robodoo commented Apr 21, 2026

Pull request status dashboard

@chga-odoo chga-odoo force-pushed the saas-19.2-real-estate-appointment-chga branch from c403915 to 3b32588 Compare April 28, 2026 06:21
@chga-odoo chga-odoo force-pushed the saas-19.2-real-estate-appointment-chga branch 3 times, most recently from 6afea83 to 42072d3 Compare May 8, 2026 09:22
@chga-odoo chga-odoo force-pushed the saas-19.2-real-estate-appointment-chga branch from 42072d3 to 8f19bec Compare May 12, 2026 12:17
Copy link
Copy Markdown
Contributor

@dhrs-odoo dhrs-odoo left a comment

Choose a reason for hiding this comment

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

Hello @chga-odoo
Thanks for your work
Few comments to adapt

Comment thread real_estate/data/ir_actions_server.xml Outdated
Comment thread real_estate/__manifest__.py
@chga-odoo chga-odoo force-pushed the saas-19.2-real-estate-appointment-chga branch 3 times, most recently from 60f7bb4 to a585658 Compare May 20, 2026 10:12
Copy link
Copy Markdown
Collaborator

@vava-odoo vava-odoo left a comment

Choose a reason for hiding this comment

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

Looks good, but is there a reason you target 19.2? (maybe a stupid question, but I don't see anything about this in the task)

@chga-odoo chga-odoo force-pushed the saas-19.2-real-estate-appointment-chga branch from a585658 to 0471107 Compare May 20, 2026 11:41
@chga-odoo chga-odoo changed the base branch from saas-19.2 to 19.0 May 20, 2026 11:42
@chga-odoo chga-odoo force-pushed the saas-19.2-real-estate-appointment-chga branch 6 times, most recently from fae4cd8 to 6718237 Compare May 22, 2026 06:35
Copy link
Copy Markdown
Collaborator

@vava-odoo vava-odoo left a comment

Choose a reason for hiding this comment

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

the timezone part is now fine, but I have a remaining question

Comment thread real_estate/data/ir_actions_server.xml Outdated
for record in records:
link = env['appointment.invite'].create({'short_code': datetime.datetime.now().strftime("%M%m%S%d%Y%H") + str(record.id), 'display_name': record.name})
env['appointment.type'].create({'name': record.name, 'display_name': record.name, 'x_property_id': record.id, 'appointment_invite_ids': [Command.link(link.id)], 'location_id': record.x_technical_partner_id.id, 'image_1920': record.image_1920, 'appointment_tz': env.ref('base.user_admin').tz or 'Europe/Brussels', 'staff_user_ids': [Command.link(record.x_agent_id.id)] if record.x_agent_id else False})
env['appointment.type'].create({'name': record.name, 'display_name': record.name, 'x_property_id': record.id, 'appointment_invite_ids': [Command.link(link.id)], 'location_id': record.x_technical_partner_id.id, 'image_1920': record.image_1920, 'appointment_tz': env.ref('base.user_admin').tz or 'Europe/Brussels', 'staff_user_ids': [Command.link(record.x_agent_id.id)] if record.x_agent_id else False, 'is_published': True, 'category': 'custom', 'event_videocall_source': False})
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

why do you publish the appointment?
I don't see this part in the spec.
The customer would select an appointment via the link, no?
Was this discussed with Loic?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sorry 😅 No need to publish this. I added it for testing purposes and forgot to remove it later 🙈

@chga-odoo chga-odoo force-pushed the saas-19.2-real-estate-appointment-chga branch from 6718237 to bd378c3 Compare May 22, 2026 09:59
- Add appointment_type demo with slots
- Set appointment defaults in server action
- Link calendar events via x_property_id
- Update event timings and timezone handling

task: 6050591
@chga-odoo chga-odoo force-pushed the saas-19.2-real-estate-appointment-chga branch from bd378c3 to 1f5dbb9 Compare May 22, 2026 11:55
@vava-odoo
Copy link
Copy Markdown
Collaborator

@robodoo r+

@fw-bot fw-bot deleted the saas-19.2-real-estate-appointment-chga branch May 29, 2026 13:18
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.

4 participants