Skip to content

Bugfix FXIOS-12835 Update asset name to fix warning #27975

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 16, 2025
Merged

Bugfix FXIOS-12835 Update asset name to fix warning #27975

merged 1 commit into from
Jul 16, 2025

Conversation

razvanlitianu
Copy link
Collaborator

@razvanlitianu razvanlitianu commented Jul 15, 2025 •
edited
Loading

📜 Tickets

Jira ticket
Github issue

💡 Description

This issue came back again so I updated the name with camel case.

🎥 Demos

Before After
Demo

📝 Checklist

  • I filled in the ticket numbers and a description of my work
  • I updated the PR name to follow our PR naming guidelines
  • I ensured unit tests pass and wrote tests for new code
  • If working on UI, I checked and implemented accessibility (Dynamic Text and VoiceOver)
  • If adding telemetry, I read the data stewardship requirements and will request a data review
  • If needed, I updated documentation and added comments to complex code
  • If needed, I added a backport comment (example @Mergifyio backport release/v120)

@razvanlitianu razvanlitianu requested a review from a team as a code owner July 15, 2025 10:38
@razvanlitianu razvanlitianu reopened this Jul 15, 2025
@razvanlitianu razvanlitianu reopened this Jul 15, 2025
@lmarceau
Copy link
Contributor

This makes me think of a similar issue I had in the past. Git doesn't commit case-sensitive changes in filenames. You could look into this thread here Let me know if that helps! 🙏

@razvanlitianu razvanlitianu reopened this Jul 16, 2025
@mobiletest-ci-bot
Copy link

Messages
📖 Edited 1 files
📖 Created 0 files

Generated by 🚫 Danger Swift against 1ffe160

@razvanlitianu razvanlitianu merged commit 51fe46d into main Jul 16, 2025
14 checks passed
@razvanlitianu razvanlitianu deleted the rlitianu/FXIOS-12835-#27974-Update-asset-name-to-fix-warning branch July 16, 2025 07:57
@razvanlitianu
Copy link
Collaborator Author

@Mergifyio backport release/v141

@mergify Mergify
Copy link
Contributor

mergify bot commented Jul 16, 2025 •
edited
Loading

backport release/v141

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Jul 16, 2025
@razvanlitianu @mergify
Bugfix FXIOS-12835 Update asset name to fix warning (#27975)
4073794 
FXIOS-12835 #27974 Update asset name to fix warning

(cherry picked from commit 51fe46d)
@mergify mergify bot mentioned this pull request Jul 16, 2025
Open
7 tasks
@razvanlitianu
Copy link
Collaborator Author

@Mergifyio backport release/v141

@mergify Mergify
Copy link
Contributor

mergify bot commented Jul 17, 2025

backport release/v141

✅ Backports have been created

@lmarceau
Copy link
Contributor

lmarceau commented Jul 18, 2025 •
edited
Loading

I still see this warning in main @razvanlitianu

Screenshot 2025-07-18 at 10 40 25 AM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FilippoZazzeroni FilippoZazzeroni FilippoZazzeroni approved these changes

@lmarceau lmarceau Awaiting requested review from lmarceau

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@razvanlitianu @lmarceau @mobiletest-ci-bot @FilippoZazzeroni