Skip to content

Commit 8c888d7

Browse files
committed
refactor(demo): move IsE2EMode to DotEnv, remove value masking
1 parent cbd847a commit 8c888d7

2 files changed

Lines changed: 9 additions & 16 deletions

File tree

examples/demo/Assets/Scripts/UI/Sections/TriggersSectionController.cs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
using System;
2+
using OneSignalDemo.Services;
23
using OneSignalDemo.ViewModels;
34
using UnityEngine.UIElements;
45

@@ -34,7 +35,7 @@ private VisualElement BuildSection()
3435
);
3536

3637
var card = SectionBuilder.CreateCard("triggers_card");
37-
if (AppViewModel.IsE2EMode)
38+
if (DotEnv.IsE2EMode)
3839
{
3940
card.style.paddingTop = 6;
4041
card.style.paddingBottom = 6;
@@ -50,7 +51,7 @@ private VisualElement BuildSection()
5051
"add_trigger_button",
5152
InvokeAdd
5253
);
53-
if (AppViewModel.IsE2EMode)
54+
if (DotEnv.IsE2EMode)
5455
{
5556
addButton.style.minHeight = 40;
5657
addButton.style.marginBottom = 4;
@@ -61,7 +62,7 @@ private VisualElement BuildSection()
6162
"add_multiple_triggers_button",
6263
InvokeAddMultiple
6364
);
64-
if (AppViewModel.IsE2EMode)
65+
if (DotEnv.IsE2EMode)
6566
{
6667
addMultipleButton.style.minHeight = 40;
6768
addMultipleButton.style.marginBottom = 4;

examples/demo/Assets/Scripts/ViewModels/AppViewModel.cs

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,6 @@ public class AppViewModel : MonoBehaviour
7373
public int LiveActivityStatusIndex => _liveActivityStatusIndex;
7474
public bool IsLiveActivityUpdating => _isLiveActivityUpdating;
7575
public bool HasApiKey => _apiService?.HasApiKey() ?? false;
76-
public static bool IsE2EMode => DotEnv.IsE2EMode;
7776

7877
public string NextStatusLabel
7978
{
@@ -97,9 +96,6 @@ public void Init(PreferencesService prefs, OneSignalApiService apiService)
9796
OneSignal.User.Changed += OnUserChanged;
9897
}
9998

100-
private static string MaskValue(string value) =>
101-
IsE2EMode && value != "—" ? new string('\u2022', value.Length) : value;
102-
10399
private void OnDestroy()
104100
{
105101
if (OneSignal.Default != null)
@@ -112,16 +108,14 @@ private void OnDestroy()
112108

113109
public void LoadInitialState()
114110
{
115-
var rawAppId = _apiService?.GetAppId() ?? "";
116-
_appId = MaskValue(rawAppId);
111+
_appId = _apiService?.GetAppId() ?? "";
117112
_consentRequired = _prefs.ConsentRequired;
118113
_privacyConsentGiven = _prefs.PrivacyConsent;
119114
_inAppMessagesPaused = _prefs.IamPaused;
120115
_locationShared = _prefs.LocationShared;
121116
_externalUserId = _prefs.ExternalUserId;
122117

123-
var rawPushId = OneSignal.User.PushSubscription.Id ?? "";
124-
_pushSubscriptionId = MaskValue(rawPushId);
118+
_pushSubscriptionId = OneSignal.User.PushSubscription.Id ?? "";
125119
_pushOptedIn = OneSignal.User.PushSubscription.OptedIn;
126120
_hasPermission = OneSignal.Notifications.Permission;
127121

@@ -582,8 +576,7 @@ public async Task FetchUserDataFromApi()
582576
_prefs.ExternalUserId = userData.ExternalId;
583577
}
584578

585-
var rawPushId = OneSignal.User.PushSubscription.Id ?? "";
586-
_pushSubscriptionId = MaskValue(rawPushId);
579+
_pushSubscriptionId = OneSignal.User.PushSubscription.Id ?? "";
587580
_pushOptedIn = OneSignal.User.PushSubscription.OptedIn;
588581
_hasPermission = OneSignal.Notifications.Permission;
589582

@@ -651,10 +644,9 @@ private static void MergeUnique(List<string> target, IEnumerable<string> source)
651644

652645
private void OnPushSubscriptionChanged(object sender, PushSubscriptionChangedEventArgs e)
653646
{
654-
var rawPushId = OneSignal.User.PushSubscription.Id ?? "";
655-
_pushSubscriptionId = MaskValue(rawPushId);
647+
_pushSubscriptionId = OneSignal.User.PushSubscription.Id ?? "";
656648
_pushOptedIn = OneSignal.User.PushSubscription.OptedIn;
657-
Debug.Log($"[{Tag}] Push subscription changed: {rawPushId}");
649+
Debug.Log($"[{Tag}] Push subscription changed: {_pushSubscriptionId}");
658650
NotifyStateChanged();
659651
}
660652

0 commit comments

Comments
 (0)