Skip to content

Conversation

@lucasromanomr
Copy link
Contributor

@lucasromanomr lucasromanomr commented Nov 4, 2025

Updates the handling of .xcmappingmodel files in the file and group element creation logic, ensuring they are treated correctly as file references rather than groups. It also adds a dedicated unit test to verify this behavior.

Before implementation, only the .xml file was visible, inside a regular folder. In addition to only being able to view the XML content and not the Xcode preview,
image (1)

Now there is interpretation via Xcode.
image

In addition to displaying correctly in the file structure, when opened, it shows the Xcode preview and not just the XML as before.

@lucasromanomr lucasromanomr requested a review from a team as a code owner November 4, 2025 15:19
@lucasromanomr lucasromanomr force-pushed the fix/show-xcmappingmodel branch from 01a666e to 5be405c Compare November 4, 2025 15:21
Copy link
Contributor

@adincebic adincebic left a comment

Choose a reason for hiding this comment

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

This is great, thanks @lucasromanomr

@lucasromanomr lucasromanomr changed the title Adds support for viewing .xcmappingmodel files. Adds support for viewing and edit .xcmappingmodel files. Nov 4, 2025
@lucasromanomr lucasromanomr changed the title Adds support for viewing and edit .xcmappingmodel files. Adds support for viewing and edit xcmappingmodel files. Nov 4, 2025
@lucasromanomr lucasromanomr changed the title Adds support for viewing and edit xcmappingmodel files. Adds support for viewing and edit xcmappingmodel files Nov 4, 2025
@lucasromanomr lucasromanomr changed the title Adds support for viewing and edit xcmappingmodel files Add support for viewing and edit xcmappingmodel files Nov 4, 2025
@lucasromanomr lucasromanomr force-pushed the fix/show-xcmappingmodel branch from 5be405c to 0e8968d Compare November 7, 2025 12:58
Copy link
Contributor

@brentleyjones brentleyjones left a comment

Choose a reason for hiding this comment

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

Please also update CHANGELOG.md.

@lucasromanomr lucasromanomr force-pushed the fix/show-xcmappingmodel branch from 0e8968d to a580f80 Compare November 12, 2025 13:58
@brentleyjones
Copy link
Contributor

You'll need to sign off on the git commit to pass the DCO check. Please mention me once you do and I can approve the PR for CI.

@lucasromanomr
Copy link
Contributor Author

mention

I did the push and I was waiting for the checks, they called me and I ended up leaving it to see later to see if everything was ok, I came back now to see and I saw. Doing it now

😅

Signed-off-by: Lucas Romano <[email protected]>
Signed-off-by: Lucas Romano <[email protected]>
@lucasromanomr lucasromanomr force-pushed the fix/show-xcmappingmodel branch from a580f80 to dcda917 Compare November 12, 2025 19:42
@brentleyjones brentleyjones merged commit cd450a8 into MobileNativeFoundation:main Nov 12, 2025
9 of 12 checks passed
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.

3 participants