-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Move pushNotificationAppID to BuildSettings #24224
Conversation
|
App Name | ![]() |
|
Configuration | Release-Alpha | |
Build Number | pr24224-b3804b0 | |
Version | 25.8 | |
Bundle ID | org.wordpress.alpha | |
Commit | b3804b0 | |
App Center Build | WPiOS - One-Offs #11681 |
|
App Name | ![]() |
|
Configuration | Release-Alpha | |
Build Number | pr24224-b3804b0 | |
Version | 25.8 | |
Bundle ID | com.jetpack.alpha | |
Commit | b3804b0 | |
App Center Build | jetpack-installable-builds #10710 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a note to say I verified the build configuration to identifier relationship for all the entries.
Next, I'll checkout locally and have a look at how they resolve, just in case.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I run the following test.
Hacked a print for the config and push id:
$ git diff
diff --git a/WordPress/Classes/ViewRelated/Me/App Settings/DebugMenuViewController.swift b/WordPress/Classes/ViewRelated/Me/App Settings/DebugMenuViewController.swift
index 99cbeb3076..103dd255de 100644
--- a/WordPress/Classes/ViewRelated/Me/App Settings/DebugMenuViewController.swift
+++ b/WordPress/Classes/ViewRelated/Me/App Settings/DebugMenuViewController.swift
@@ -1,5 +1,6 @@
import UIKit
import AutomatticTracks
+import BuildSettingsKit
import SwiftUI
import WordPressFlux
import TipKit
@@ -9,6 +10,16 @@ struct DebugMenuView: View {
fileprivate var navigation: NavigationContext
+ private var name: String {
+#if DEBUG
+ "Debug"
+#elseif ALPHA_BUILD
+ "Alpha"
+#else
+ "Release"
+#endif
+ }
+
var body: some View {
List {
Section { main }
@@ -17,6 +28,9 @@ struct DebugMenuView: View {
Section(Strings.sectionTipKit) { tipKit }
}
Section(Strings.sectionLogging) { logging }
+ Section("Push info") {
+ Text("On \(name) build with PushNotificationAppID: \(BuildSettings.pushNotificationAppID)")
+ }
}
.toolbar {
ToolbarItem(placement: .principal) {
diff --git a/WordPress/WordPress.xcodeproj/xcshareddata/xcschemes/Jetpack.xcscheme b/WordPress/WordPress.xcodeproj/xcshareddata/xcschemes/Jetpack.xcscheme
index 5e3b091eb1..6b60b8d45e 100644
--- a/WordPress/WordPress.xcodeproj/xcshareddata/xcschemes/Jetpack.xcscheme
+++ b/WordPress/WordPress.xcodeproj/xcshareddata/xcschemes/Jetpack.xcscheme
@@ -59,7 +59,7 @@
</Testables>
</TestAction>
<LaunchAction
- buildConfiguration = "Debug"
+ buildConfiguration = "Release"
selectedDebuggerIdentifier = "Xcode.DebuggerFoundation.Debugger.LLDB"
selectedLauncherIdentifier = "Xcode.DebuggerFoundation.Launcher.LLDB"
launchStyle = "0"
However, for some reason I can't explain both Release and Release-Alpha builds crash on my device 🤔
I even let CI build the alpha build, but the result was the same.
I really want to get to the bottom of this, but I already spent some time running around in circles. I'm sure it's something dumb that I'm doing that I'll figure out later after taking a step back. I want to move on with the #24165 work.
Approving to unblock.
Changes
org.wordpress.internal
) as discussedTo test:
Regression Notes
Potential unintended areas of impact
What I did to test those areas of impact (or what existing automated tests I relied on)
What automated tests I added (or what prevented me from doing so)
PR submission checklist:
RELEASE-NOTES.txt
if necessary.Testing checklist: