[Playground] [Frontend] Migrate Playground to Google Analytics 4 #24957

nausharipov · 2023-01-10T11:54:11Z

Resolves #24924
Resolves #24901
Resolves #25466
Resolves #25489
Resolves #24591

This PR replaces the old Google Analytics with the new Google Analytics 4 in Playground. It is out of the Playground scope, but this is required for Tour of Beam, and it is easier to maintain only a single version of Analytics in both projects.
In this PR only Playground is migrated, and for ToB some TODOs are inserted. They will be resolved in

[Task]: Google Analytics 4 in Tour of Beam #25466
Some work on ToB is there because we initially wanted to work only on it and keep Playground intact, and it is easier to not cut that ToB work now.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
Update CHANGES.md with noteworthy changes.
If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

AnalyticsEvent and AnalyticsService in PGC, feedback split, GetIt instead of Provider

alexeyinkin · 2023-01-10T13:12:13Z

playground/frontend/lib/locator.dart

@@ -57,3 +59,7 @@ void _initializeState() {
    ),
  );
 }
+
+void _initializeAnalyticsService() {


Suggested change

void _initializeAnalyticsService() {

void _initializeServices() {

alexeyinkin · 2023-01-10T13:17:45Z

playground/frontend/lib/modules/analytics/service.dart

-
-  static AnalyticsService get(BuildContext context) {
-    return Provider.of<AnalyticsService>(context, listen: false);
+abstract class AnalyticsService extends GeneralAnalyticsService {


How about this?

AnalyticsService -> PlaygroundAnalyticsService -> PlaygroundGoogleAnalyticsService -> TobAnalyticsService -> TobGoogleAnalyticsService

Simplified back to to AnalyticsService -> GoogleAnalyticsService.

alexeyinkin · 2023-01-10T13:20:59Z

playground/frontend/lib/locator.dart

@@ -57,3 +59,7 @@ void _initializeState() {
    ),
  );
 }
+
+void _initializeAnalyticsService() {
+  GetIt.instance.registerSingleton(GoogleAnalyticsService());


You can register a single object under two handles:

Suggested change

GetIt.instance.registerSingleton(GoogleAnalyticsService());

GetIt.instance.registerSingleton<AnalyticsService>(service);

GetIt.instance.registerSingleton<PlaygroundAnalyticsService>(service);

This way playground_components will request <AnalyticsService> and be able to submit generic events, while Playground and ToB will fetch their own subclasses to submit their specific events.

alexeyinkin · 2023-01-10T13:26:13Z

...und/frontend/lib/pages/standalone_playground/widgets/feedback/feedback_dropdown_content.dart

+                          AnalyticsService.get().trackClickSendPositiveFeedback(
+                            text,
+                          );
+                        } else {


Second thoughts, this is too much of investment into boolean. I suggest making an enum FeedbackRating with positive and negative.

alexeyinkin · 2023-01-26T04:59:09Z

learning/tour-of-beam/frontend/lib/modules/analytics/google_analytics_service.dart

+
+import 'package:get_it/get_it.dart';
+import 'package:playground_components/playground_components.dart';
+import 'package:usage/usage_html.dart';


Is there a way to use a platform-agnostic implementation?

The usage package will not be used, because it doesn't support GA4.

Dropped usage, manually submitting to GA4.

alexeyinkin · 2023-01-26T05:03:31Z

learning/tour-of-beam/frontend/lib/modules/analytics/google_analytics_service.dart

+      AnalyticsEvent(
+        action: TobAnalyticsEvents.closeUnit,
+        category: TobAnalyticsCategories.unit,
+        label: '${sdk.title}_${unitId}_${timeSpent.inSeconds}s',


Is there a way to send raw property values for sdk.title, unit.id, and seconds to filter the events later?

The usage package will not be used, because it doesn't support GA4.

Dropped usage, manually submitting to GA4.

alexeyinkin · 2023-01-26T05:05:47Z

learning/tour-of-beam/frontend/lib/modules/analytics/service.dart

+  Future<void> openUnit(Sdk sdk, UnitModel unit);
+  Future<void> closeUnit(Sdk sdk, String unitId, Duration timeSpent);
+  Future<void> completeUnit(Sdk sdk, UnitModel unit);
+  // TODO(nausharipov): implement


Deleted completeModule after the discussion with Oleh.

alexeyinkin · 2023-01-26T05:09:28Z

learning/tour-of-beam/frontend/lib/pages/tour/state.dart

@@ -57,6 +62,7 @@ class TourNotifier extends ChangeNotifier with PageStateMixin<void> {
    _appNotifier.addListener(_onAppNotifierChanged);
    _authNotifier.addListener(_onUnitProgressChanged);
    _onUnitChanged();
+    window.onBeforeUnload.listen(_onTabClosed);


Make a CloseNotifier and do not import dart:html here?

Made WindowCloseNotifier.

alexeyinkin · 2023-01-26T06:13:13Z

learning/tour-of-beam/frontend/lib/config.dart

@@ -26,7 +26,8 @@ const cloudFunctionsBaseUrl = 'https://'

 // Copied from Playground's config.g.dart

-const String kAnalyticsUA = 'UA-73650088-2';
+const String kAnalyticsUA = 'G-349612187';


Does the usage package work with the Analytics-4 new counters?

The open issue indicates that it doesn't. I haven't found a wrapper package that supports GA4 other than firebase_analytics. Shall I use firebase_analytics?

Dropped usage, manually submitting to GA4.

alexeyinkin · 2023-01-26T06:27:31Z

playground/frontend/playground_components/lib/src/services/analytics/generic/events.dart

+
+class GenericAnalyticsEvents {
+  static const run = 'click_run';
+  // TODO(nausharipov): rename example to snippet everywhere?


I suggest keeping it to not break the continuity of statistics.

Renamed since we break continuity anyway with the new GA version.

alexeyinkin · 2023-01-26T06:29:01Z

...ound/frontend/playground_components/lib/src/services/analytics/google_analytics_service.dart

+
+  @override
+  AnalyticsEvent? lastSentEvent;
+


There should be more events common for the two projects. Cancelling comes to mind.

I suggest also adding feedback here. This way we make sure that both ToB and Playground handle the feedback the same way.

alexeyinkin · 2023-01-26T06:32:05Z

playground/frontend/playground_components/lib/src/widgets/run_or_cancel_button.dart

+        await AnalyticsService.get().trackRunExample(
+          playgroundController.selectedExample!.name,
+        );


Maybe not await most of the events? Except maybe feedback because the event is the business logic there.

alexeyinkin · 2023-01-26T06:33:11Z

playground/frontend/lib/pages/standalone_playground/widgets/feedback/rating_enum.dart

+ * limitations under the License.
+ */
+
+enum FeedbackRating { positive, negative }


Trailing comma, auto-format.

alexeyinkin · 2023-01-26T06:35:08Z

playground/frontend/lib/pages/standalone_playground/widgets/feedback/playground_feedback.dart

  _setEnjoying(BuildContext context, bool isEnjoying) {
    return () {
      _getFeedbackState(context, false).setEnjoying(isEnjoying);
-      AnalyticsService.get(context).trackClickEnjoyPlayground(isEnjoying);
+      PlaygroundAnalyticsService.get().trackClickEnjoyPlayground(isEnjoying);


Use FeedbackRating everywhere instead of bool?

alexeyinkin · 2023-02-14T13:16:32Z

playground/frontend/Dockerfile

@@ -28,6 +28,7 @@ RUN apt-get update && apt-get install -y wget xz-utils git
 RUN wget https://storage.googleapis.com/flutter_infra_release/releases/stable/linux/flutter_linux_$FLUTTER_VERSION-stable.tar.xz &&\
    tar -xf flutter_linux_$FLUTTER_VERSION-stable.tar.xz &&\
    mv flutter /opt/ &&\
+    git config --global --add safe.directory /opt/flutter &&\


The recent version of Debian 11.6 has newer git that by default does not allow operations outside of home directories. Flutter uses git to download dependencies when it is ran for the first time, so this is required.

alexeyinkin · 2023-02-14T13:19:30Z

playground/frontend/lib/components/dropdown_button/dropdown_button.dart

-                    child: widget.createDropdown(_close),
+                    child: child,


chlid had been creating multiple times here by OverlayEntry builder leading to false positive analytics events.

…lytics

github-actions · 2023-02-17T20:35:30Z

Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment assign set of reviewers

…apache#24924)

* tob event context (apache#25466) * flutter v3.7.3 upgrade (apache#25466) * BeamRunShortcut (apache#25466) * comment fixes (apache#25466) * tob app name for analytics parameter (apache#25466) * comment fixes (apache#25466) * the rest of tob events except for example modified (apache#25466) * questioned print (apache#25466) * additionalParams in SnippetModifiedAnalyticsEvent (apache#25466) * comment fixes & renamed events to past tense (apache#25466) * comment fixes (apache#25466) * reportIssueClicked (apache#25466) * sort BeamAnalyticsEvents (apache#25466) --------- Co-authored-by: darkhan.nausharipov <darkhan.nausharipov@kzn.akvelon.com>

damondouglas

Change request is more for increased test coverage. Analytics is important as we assess the value of Beam Playground and Tour of Beam as our onboarding tool for the community.
Comments are more code design related and subjective. Otherwise, I was able to run locally and tests passed.

damondouglas · 2023-03-08T22:32:39Z

learning/tour-of-beam/frontend/lib/config.dart

+const String kAnalyticsUA = 'G-BXFP2FNCKC';
+
+const String kApiClientURL = 'https://router.play-dev.beam.apache.org';
+const String kApiJavaClientURL = 'https://java.play-dev.beam.apache.org';
+const String kApiGoClientURL = 'https://go.play-dev.beam.apache.org';
+const String kApiPythonClientURL = 'https://python.play-dev.beam.apache.org';
+const String kApiScioClientURL = 'https://scio.play-dev.beam.apache.org';


Code comment where these come from and how they are derived? Someone should read this and know how to change if needed.

Just two lines above it reads // Copied from Playground's config.g.dart which in turn has an explanation.

config.g.dart will be gone after merging #25610
If it happens before this merge, we will delete the ToB's config.dart when merging master here. Otherwise we will update #25610 accordingly if this is merged first.

damondouglas · 2023-03-08T22:33:21Z

learning/tour-of-beam/frontend/lib/locator.dart

@@ -61,3 +64,12 @@ void _initializeState() {
    ),
  );
 }
+
+void _initializeServices() {
+  final googleAnalyticsService = GoogleAnalytics4Service(


What if the analytics service changes again from GoogleAnalytics4Service to something else?

Also, is it possible to alias the library imports so someone knows where the class is from? So instead of SomeClass(), we have library.SomeClass()? Yes, in an IDE one could navigate to definition but many times people are reading code on GitHub directly; it's symbol lookup doesn't always work.

Renamed to BeamAnalyticsService and BeamGoogleAnalyticsSevice. I like it better then library.Class because the latter is rarely used, stands out, and questions why not import everything like that. And then we lose the Dart simplicity.

I. In that case we will make BeamSomethingElseAnalyticsService extends BeamAnalyticsService, instantiate it here and register it in GetIt under the same BeamAnalyticsService interface. Renamed the variable to analyticsService to further simplify the migration.

damondouglas · 2023-03-08T22:33:54Z

learning/tour-of-beam/frontend/lib/locator.dart

+  GetIt.instance.registerSingleton<AnalyticsService>(
+    googleAnalyticsService,
+  );


Instead of using GetIt why not just reference googleAnalyticsService? The project claims

As your App grows, at some point you will need to put your app's logic in classes that are separated from your Widgets. Keeping your widgets from having direct dependencies makes your code better organized and easier to test and maintain.

It isn't clear from this what is actually implementing the abstract AnalyticsService class without some digging. The promise to make the code better organized, at least in my attempt at reading the code, didn't seem to happen.

Although the analytics classes are inside the shared playground_components package, the instantiation of the service is external to it. This gives us the future flexibility to:

Change the implementations for Playground and ToB independently.

Write tests that mock the analytics service.

Use the shared Playground widgets in a new app that does not send analytics data by registering a null-object for analytics.

There are a number of ways to implement this:

Wrap Playground widgets in a Provider widget to provide them with a specific analytics service. But this gives no compile-time guarantee that the provider is in the tree and leads to bugs.

Use a service locator of which get_it is the most popular one. So we did.

It isn't clear from this what is actually implementing the abstract AnalyticsService class without some digging.

We need easy swap to a potential SomethingElseAnalyticsService, so this abstraction is on purpose. We could register the service to be read as get<GoogleAnalyticsService>(), but then swapping would affect dozens of locations.

The promise get_it gives is that widgets no longer have to reference global variables or rely on Providers. And this promise is fulfilled. Provider leads to even more digging while global variables bar the 3 opportunities above.

damondouglas · 2023-03-08T22:39:00Z

learning/tour-of-beam/frontend/lib/models/event_context.dart

+class TobEventContext with EquatableMixin {
+  const TobEventContext({
+    required this.sdkId,
+    required this.unitId,
  });

+  final String? sdkId;
+  final String? unitId;
+
+  static const empty = TobEventContext(
+    sdkId: null,
+    unitId: null,
+  );


Could we spell out Tob to TourOfBeam and it isn't clear why it needs to implement EquatableMixin? Is it because there are two nullable properties and doing equals on this is challenging?

I would rather keep it Tob because we have many classes that need to emphasize their belonging to TourOfBeam. With this many classes we may assume the reader will soon be familiar with Tob. Some of the classes are long already. Some of them will become ambiguous if expanded, like is TourOfBeamMarkdown a 'Tour of BeamMarkdown'?

We need to compare objects in tests. EqualityMixin takes care of both operator == and hashCode.

damondouglas · 2023-03-08T22:40:54Z

...ound/frontend/lib/modules/editor/components/share_dropdown/share_tabs/share_format_enum.dart

+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+


I realize that the enum is obvious from its members but could we see a code comment describing its purpose which I'm assuming is in the context of Analytics events?

damondouglas · 2023-03-09T00:20:59Z

playground/frontend/lib/modules/actions/components/new_example_action.dart

@@ -38,7 +38,9 @@ class NewExampleAction extends StatelessWidget {
        label: 'intents.playground.newExample'.tr(),
        onPressed: () {
          launchUrl(Uri.parse('/'));
-          AnalyticsService.get(context).trackClickNewExample();
+          PlaygroundComponents.analyticsService.sendUnawaited(


I agree with @Malarg here that sendNewExampleEvent makes sense instead of sendUnawaited.

damondouglas · 2023-03-09T00:27:11Z

playground/frontend/lib/services/analytics/events/loaded.dart

+import 'constants.dart';
+
+/// Playground is loaded.
+class LoadedAnalyticsEvent extends AnalyticsEvent {


When you change this PR to add more tests, it would be nice to see this included. It looks like this and its base AnalyticsEvent class are just value classes without any exciting functionality so perhaps we can see coverage in the context of other analytics tests such as a test that loads the Playground and asserts that the LoadedAnalyticsEvent has the assigned sdk and/or snippet.

damondouglas · 2023-03-09T00:30:29Z

playground/frontend/lib/services/analytics/events/new_example.dart

+import 'constants.dart';
+
+/// 'New Example' button is clicked in the app header.
+class NewExampleAnalyticsEvent extends AnalyticsEvent {


It would be great to see more test coverage for this. Like other extensions of AnalyticsEvent seemingly just data classes, it would be great to assert that the values are what is expected (I see the name property for example). In this context I would like to see a test mocking New Example button clicked and that the assertion passes that the name is what it should be.

damondouglas · 2023-03-09T00:33:51Z

playground/frontend/playground_components/lib/src/services/analytics/analytics_service.dart

+  }
+
+  static AnalyticsService get() {
+    return GetIt.instance.get<AnalyticsService>();


I don't see how GetIt is making it clear where the actual implementation exists. I had to search quite a bit until I found the actual implementation. Nonetheless, if we decide to keep and use GetIt, can we make good use of it and expand our test coverage as the package promises to make easier?

It does not because its goal is to decouple usage from implementation.

A reader familiar with GetIt would find-in-files 'GetIt.instance.register' to get to implementations.

As for swapping in mocks in tests, GetIt can do that for unit tests because there we can have different initialization. But we aim to not add new mocks to unit tests because we plan to migrate to another mocking library in the next phase.

Instead we test analytics with integration tests (since we anyway test everything by clicking through), and there we do not have the control of implementations because the real-app implementations are loaded. Thus we check BeamAnalyticsService.lastEvent.

The only real benefit of GetIt that we use for now is to make playground_components unaware of the implementation without using Provider and global variables, which justifies it to us.

damondouglas · 2023-03-09T00:36:43Z

playground/frontend/playground_components/lib/src/services/analytics/analytics_service.dart

+  }
+
+  /// Sends [event] asynchronously without returning a [Future].
+  void sendUnawaited(AnalyticsEvent event) {


Why can't we just have Future<void> send(AnalyticsEvent event) async { ... }?

Because when we call it and discard the future, we get a discarded_future lint. We would have to do one of the following with it:

Disable the lint project-wide, which we do not want to, because it guards from logic errors.

Disable it at each occurrence with which is harder to read.

…lytics

…vigation (apache#24924) (#444) * Test analytics events for feedback, report issue, and external url navigation (apache#24924) * Test for run_started, run_finished, run_cancelled, loaded, sdk_selected, snippet_selected events (apache#24924) * Fix a run test (apache#24924) * Fix a test (apache#24924) * Fix a test (apache#24924) * Test for theme_set, new_example, and shortcuts analytics events (apache#24924) * Test code sharing (apache#24924) * Test for snippet_modified and snippet_reset events (apache#24924) * Clean up (apache#24924) * Clean up (apache#24924)

…lytics

damondouglas

@damccorm LGTM

darkhan.nausharipov added 3 commits January 10, 2023 17:18

removed goToContextLine

daa56b5

PG analytics refactoring (apache#24901) (apache#24924):

e143de9

AnalyticsEvent and AnalyticsService in PGC, feedback split, GetIt instead of Provider

AnalyticsCategories, AnalyticsEvents (apache#24924)

6ae97ef

github-actions bot added the learning label Jan 10, 2023

alexeyinkin reviewed Jan 10, 2023

View reviewed changes

darkhan.nausharipov added 13 commits January 10, 2023 21:49

comment fixes (apache#24924)

50adca2

renamed playground demo widget (apache#24924)

74531cb

tob analytics, generic analytics, events, categories (apache#24924)

50b1fba

deleted GenericAnalyticsService (apache#24924)

61a7764

trackRunExample usage (apache#24924)

5057195

added events (apache#24924)

a3b8460

Merge branch 'master' into issue24924-tobf-analytics

a8316c3

new pubspec.lock (apache#24924)

d29eb80

open and complete unit event implementations (apache#24924)

de999f1

time spent in unit on close (apache#24924)

5981c6b

on before load to track time on close (apache#24924)

9b67020

_onTabClosed (apache#24924)

324137f

Merge master

866bbb1

nausharipov requested a review from alexeyinkin January 24, 2023 06:00

alexeyinkin suggested changes Jan 26, 2023

View reviewed changes

comment fixes (apache#24924)

e2e270d

nausharipov requested a review from alexeyinkin February 3, 2023 08:53

darkhan.nausharipov and others added 8 commits February 6, 2023 19:47

Merge branch 'master' into issue24924-tobf-analytics

6084e18

comment fixes (apache#24924)

5b06e3f

made test tasks work (apache#24924)

3c481b3

Use Google Analytics 4 JS in Playground (apache#24924)

c007d6f

Update staging URLs and Google Analytics property ID (apache#24924)

4c1932e

Fix Playground integation tests (apache#24924)

c496f09

Disable old Tour of Beam analytics (apache#24924)

e68e03c

Clean up (apache#24924)

004de5d

alexeyinkin reviewed Feb 14, 2023

View reviewed changes

Clean up (apache#24924)

71d26ac

nausharipov changed the title ~~[ToB] [Frontend] Tour of Beam analytics~~ [Playground] [Frontend] Migrate Playground to Google Analytics 4 Feb 14, 2023

alexeyinkin approved these changes Feb 14, 2023

View reviewed changes

nausharipov marked this pull request as ready for review February 14, 2023 14:17

Merge remote-tracking branch 'origin/master' into issue24924-tobf-ana…

ea7c165

…lytics

alexeyinkin and others added 5 commits February 22, 2023 16:49

Use the production Google Analytics property, rename event parameters (…

bbd7024

…apache#24924)

Add LoadedAnalyticsEvent (apache#24924)

ec1f7df

Fix (apache#24924)

45a1fc3

Fix Run hot key (apache#24924)

ed1fac5

damondouglas requested changes Mar 9, 2023

View reviewed changes

alexeyinkin and others added 8 commits March 9, 2023 17:58

Address review comments (apache#24924)

385132c

Added comments (apache#24924)

cd79651

Added comments (apache#24924)

fb34d74

Add comments (apache#24924)

183fb61

Merge remote-tracking branch 'apache/master' into issue24924-tobf-ana…

df7700c

…lytics

Fix SDK switching (apache#24924)

08f05a8

Fix integration tests (apache#24924)

2ca8e2c

alexeyinkin mentioned this pull request Mar 22, 2023

[Task]: Do not URL-encode links when sharing a catalog example in Playground #25489

Closed

15 tasks

alexeyinkin added 3 commits March 28, 2023 17:38

Merge remote-tracking branch 'apache/master' into issue24924-tobf-ana…

2966768

…lytics

Fix tests (apache#24924)

fbefe85

Differentiate production and dev analytics (apache#24924)

9eb41e7

damondouglas approved these changes Apr 12, 2023

View reviewed changes

damccorm approved these changes Apr 12, 2023

View reviewed changes

damccorm merged commit 2597ca1 into apache:master Apr 12, 2023

	void _initializeAnalyticsService() {
	void _initializeServices() {

	GetIt.instance.registerSingleton(GoogleAnalyticsService());
	GetIt.instance.registerSingleton<AnalyticsService>(service);
	GetIt.instance.registerSingleton<PlaygroundAnalyticsService>(service);

[Playground] [Frontend] Migrate Playground to Google Analytics 4 #24957

[Playground] [Frontend] Migrate Playground to Google Analytics 4 #24957

Conversation

nausharipov commented Jan 10, 2023 • edited Loading

GitHub Actions Tests Status (on master branch)

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

nausharipov Feb 2, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

github-actions bot commented Feb 17, 2023

damondouglas left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

alexeyinkin Mar 9, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

alexeyinkin Mar 9, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

alexeyinkin Mar 9, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

damondouglas left a comment

Choose a reason for hiding this comment

nausharipov commented Jan 10, 2023 •

edited

Loading

nausharipov Feb 2, 2023 •

edited

Loading

damondouglas left a comment •

edited

Loading

alexeyinkin Mar 9, 2023 •

edited

Loading

alexeyinkin Mar 9, 2023 •

edited

Loading

alexeyinkin Mar 9, 2023 •

edited

Loading