Skip to content
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

chore: convert to folder Wire-iOS - WPB-16244 #2421

Open
wants to merge 16 commits into
base: develop
Choose a base branch
from

Conversation

netbe
Copy link
Collaborator

@netbe netbe commented Jan 24, 2025

TaskWPB-16244 [iOS] Convert Wire-iOS groups to folder

Issue

  • Convert groups to folders for Wire-iOS
  • Deleted unused files

Only a supporting Files group remains for Wire-iOS

Testing

  • should pass CI

Checklist

  • Title contains a reference JIRA issue number like [WPB-XXX].
  • Description is filled and free of optional paragraphs.
  • Adds/updates automated tests.

UI accessibility checklist

If your PR includes UI changes, please utilize this checklist:

  • Make sure you use the API for UI elements that support large fonts.
  • All colors are taken from WireDesign.ColorTheme or constructed using WireDesign.BaseColorPalette.
  • New UI elements have Accessibility strings for VoiceOver.

@netbe netbe requested a review from caldrian January 24, 2025 12:15
@caldrian caldrian changed the title chore: convert to folder Wire-iOS - chore: convert to folder Wire-iOS - WPB-15572 Jan 24, 2025
@echoes-hq echoes-hq bot added the echoes: technical-roadmap Work contributing to the Technical Roadmap, to improve our velocity or reduce the technical debt. label Jan 24, 2025
@caldrian caldrian closed this Jan 24, 2025
@caldrian caldrian deleted the chore/convert-to-folder-wire-ios branch January 24, 2025 13:14
@caldrian caldrian restored the chore/convert-to-folder-wire-ios branch January 24, 2025 13:15
@caldrian caldrian reopened this Jan 24, 2025
@netbe netbe added the WIP label Jan 31, 2025
@netbe netbe force-pushed the chore/convert-to-folder-wire-ios branch from 0d66ca5 to 754a668 Compare February 21, 2025 17:02
@@ -14,111 +14,111 @@

files: [
{
"source" : "/wire-ios/Wire-iOS/Resources/Base.lproj/Localizable.strings",
"source" : "/wire-ios/Wire-iOS/Resources/Localization/Base.lproj/Localizable.strings",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I had to move the localizable inside a folder, hence the change

<key>NSExtensionPointIdentifier</key>
<string>com.apple.share-services</string>
<key>NSExtensionPrincipalClass</key>
<string>$(PRODUCT_MODULE_NAME).ShareExtensionViewController</string>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this replaces the MainInterface storyboard

@netbe netbe marked this pull request as ready for review February 21, 2025 17:31
@netbe netbe requested review from a team and removed request for a team February 21, 2025 17:36
@netbe netbe changed the title chore: convert to folder Wire-iOS - WPB-15572 chore: convert to folder Wire-iOS - WPB-16244 Feb 21, 2025
Copy link
Contributor

Test Results

    1 files      2 suites   1m 57s ⏱️
1 789 tests 1 763 ✅ 26 💤 0 ❌
1 799 runs  1 773 ✅ 26 💤 0 ❌

Results for commit 9f2e0b0.

@datadog-wireapp
Copy link

Datadog Report

Branch report: chore/convert-to-folder-wire-ios
Commit report: 904fcfc
Test service: wire-ios-mono

✅ 0 Failed, 1687 Passed, 26 Skipped, 1m 57.66s Total Time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
echoes: technical-roadmap Work contributing to the Technical Roadmap, to improve our velocity or reduce the technical debt. WIP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants