Skip to content

[App size reduction] Remove shipping image duplication #15788

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

Open
wants to merge 2 commits into
base: trunk
Choose a base branch
from

Conversation

iamgabrielma
Copy link
Contributor

@iamgabrielma iamgabrielma commented Jun 20, 2025

Description

Part of WOOMOB-534 to reduce app size.

This PR removes woo-shipping-label-printing-instructions image, which is a duplicated of woo-shipping-label-creation, already declared in UIImage+Woo. This is used in:

  • ShippingLabelPrintingInstructionsView
  • PrintCustomsFormsView
  • ShippingLabelCreationInfoViewController
Before After
Screenshot 2025-06-20 at 15 44 17 Screenshot 2025-06-20 at 15 45 40

Testing information

I've just tested in the simulator preview, I've tried with adding shipping labels to an order but I'm unsure how to trigger these flows 😅

The same image is used for woo-shipping-label-printing-instructions and woo-shipping-label-creation, which is already in Assets
@dangermattic
Copy link
Collaborator

dangermattic commented Jun 20, 2025

1 Warning
⚠️ This PR is larger than 300 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.

Generated by 🚫 Danger

@iamgabrielma iamgabrielma added type: enhancement A request for an enhancement. type: task An internally driven task. feature: shipping labels Related to creating, ordering, or printing shipping labels. labels Jun 20, 2025
@iamgabrielma iamgabrielma added this to the 22.7 milestone Jun 20, 2025
@iamgabrielma iamgabrielma marked this pull request as ready for review June 20, 2025 08:46
@wpmobilebot
Copy link
Collaborator

App Icon📲 You can test the changes from this Pull Request in WooCommerce iOS Prototype by scanning the QR code below to install the corresponding build.

App NameWooCommerce iOS Prototype
Build Number30616
VersionPR #15788
Bundle IDcom.automattic.alpha.woocommerce
Commita2fc8b4
Installation URL42qte8lkklfdo
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: shipping labels Related to creating, ordering, or printing shipping labels. type: enhancement A request for an enhancement. type: task An internally driven task.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants