Skip to content

Conversation

vishal-singh-webkul
Copy link
Contributor

No description provided.

@vishal-singh-webkul vishal-singh-webkul changed the title Added: Hotel address and removed hotel column from the room type table in the booking invoice. Added: Added Hotel address and removed hotel column from the room type table in the booking invoice. May 7, 2025
@rohit053 rohit053 requested a review from Copilot May 15, 2025 06:34
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates invoice templates to remove the hotel column from the room details table and integrate hotel address details into the invoice display and PDF content for a more comprehensive invoice presentation.

  • Removed the hotel column from the room details table and updated the corresponding colspan values.
  • Added a hotel detail section in the invoice addresses tab.
  • Updated PDF content generation to include formatted hotel address information.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
pdf/invoice.product-tab.tpl Removed the hotel column and updated colspan values accordingly.
pdf/invoice.addresses-tab.tpl Added a hotel detail section to display the hotel address.
classes/pdf/HTMLTemplateInvoice.php Integrated hotel address formatting and added the hotel address to the invoice data.
Comments suppressed due to low confidence (1)

pdf/invoice.product-tab.tpl:30

  • After removing the hotel column, please verify that the updated colspan values maintain consistent alignment across all table rows under different conditions.
<th colspan="{if $display_product_images}8{else}7{/if}" class="header">{l s='Rooms Details' pdf='true'}</th>

@rohit053 rohit053 merged commit 0b21920 into Qloapps:develop May 15, 2025
@vishal-singh-webkul vishal-singh-webkul deleted the gli-2581 branch June 19, 2025 05:30
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