EA-219: Update to OpenMRS Platform 2.8.x #263

wikumChamith · 2025-07-16T14:54:07Z

Ticket: https://openmrs.atlassian.net/browse/EA-219

Legacy branch: https://github.com/openmrs/openmrs-module-emrapi/tree/2.x

wikumChamith · 2025-07-22T05:29:53Z

@dkayiwa, why are we seeing a forbidden class exception even though the MetadataPackagesConfig class is on the whitelisting list?

openmrs-module-emrapi/omod/src/main/resources/config.xml

Lines 151 to 155 in 5d0e79d

    
           <globalProperty> 
        
               <property>emrapi.serializer.whitelist.types</property> 
        
               <defaultValue>org.openmrs.module.emrapi.metadata.MetadataPackagesConfig,org.openmrs.module.emrapi.metadata.MetadataPackageConfig</defaultValue> 
        
               <description></description> 
        
           </globalProperty>

mseaton · 2025-07-22T13:32:08Z

api/src/main/java/org/openmrs/module/emrapi/account/provider/CoreProviderService.java

+    public void setProviderRole(Provider provider, ProviderRole providerRole) {
+        provider.setProviderRole(providerRole);
    }
 }


We should be able to delete this class, no?

mseaton · 2025-07-22T13:32:34Z

api/src/main/java/org/openmrs/module/emrapi/account/provider/ProviderServiceFacade.java


-    void setProviderRole(Provider provider, Object providerRole);
+    void setProviderRole(Provider provider, ProviderRole providerRole);
 }


We should be able to delete this class, no?

mseaton · 2025-07-22T13:36:41Z

api/src/test/java/org/openmrs/module/emrapi/account/AccountDomainWrapperTest.java

 import static org.openmrs.util.OpenmrsConstants.USER_PROPERTY_LOGIN_ATTEMPTS;

-public class AccountDomainWrapperTest extends EmrApiContextSensitiveTest {
+public class AccountDomainWrapperTest extends BaseModuleContextSensitiveTest {


If you are going to switch from EmrApiContextSensitiveTest to BaseModuleContextSensitiveTest, why aren't you deleting the (now empty) EmrApiContextSensitiveTest. Any reason not to keep it around and use it even if it is empty? Gives a convenient place to put code that you need to execute for every test in this module.

mseaton · 2025-07-22T13:37:13Z

api/src/test/java/org/openmrs/module/emrapi/account/AccountDomainWrapperTest.java

-        providerManagementService = mock(ProviderManagementService.class);
-        providerServiceFacade = new MockProviderServiceFacade(new ProviderManagementProviderService(providerService, providerManagementService));
+        providerManagementService = mock(ProviderService.class);
+        providerServiceFacade = new MockProviderServiceFacade(new CoreProviderService(providerService));


This entire providerServiceFacade idea is no longer needed. Just remove it and use providerService directly

mseaton · 2025-07-22T13:37:40Z

api/src/test/java/org/openmrs/module/emrapi/account/AccountDomainWrapperTest.java


        providerIdentifierGenerator = mock(ProviderIdentifierGenerator.class);
-        when(providerIdentifierGenerator.generateIdentifier(any(Provider.class))).thenReturn("456");
+        lenient().when(providerIdentifierGenerator.generateIdentifier(any(Provider.class))).thenReturn("456");


What do these lenient() calls do?

mseaton · 2025-07-22T13:41:53Z

omod/pom.xml

-					<argLine>-Xmx512m -XX:MaxPermSize=512m -Djdk.net.URLClassPath.disableClassPathURLCheck=true</argLine>
-					<includes>
-						<include>**/*Test.java</include>
-						<include>**/*_IT.java</include>


Just want to confirm that after removing this that all tests are still executed (eg all _IT tests)

wikumChamith · 2025-07-23T07:33:51Z

@mseaton I updated the PR according to your suggestions. Do you know why I am getting the xstream.security.ForbiddenClassException in MetadataUtilTest ?

wikumChamith · 2025-07-23T10:51:40Z

Should I remove the MetadataUtilTest, since MetadataUtil has been deprecated and its functionality has been moved to the Metadata Sharing module?

mseaton · 2025-07-23T16:10:17Z

Should I remove the MetadataUtilTest, since MetadataUtil has been deprecated and its functionality has been moved to the Metadata Sharing module?

I think this is fine @wikumChamith . If you do so, please also remove MetadataUtil. It has been deprecated for a long time, and in any event this new change you are introducing definitely requires a new major version anyway.

So please update this PR to change the version in the pom to 3.0.0-SNAPSHOT.

(I am doing a release from the master branch now which will be 2.4.0, and we can fork off from there as needed as a maintenance branch for those that cannot move to 2.8.x in the near term)

mseaton

LGTM (though I don't know about the changes to the findPatient DAO, I didn't review these in detail and don't have a good sense of why they are required)

wikumChamith · 2025-07-27T06:26:05Z

LGTM (though I don't know about the changes to the findPatient DAO, I didn't review these in detail and don't have a good sense of why they are required)

PatientSearchCriteria has been deprecated, and this was showing some errors. So, I decided to replace it with the JPA Criteria API, which is the recommended approach starting from Platform 2.7.0.

dkayiwa · 2025-07-27T23:25:59Z

@wikumChamith did you get a chance to look at the build errors?

wikumChamith · 2025-07-28T04:20:24Z

@wikumChamith did you get a chance to look at the build errors?

@dkayiwa Looks like these errors are coming from core, specifically after this commit: openmrs/openmrs-core@ec1f42c

cc: @rkorytkowski

rkorytkowski · 2025-07-28T08:13:58Z

The issue is TestingApplicationContext overrides sessionFactory from core and doesn't set cacheConfig property. I think I need to move cacheConfig to be autowired again in case of this kind of overwrite... Overwriting bean definition is hacky and we will want to come up with some better approach here. I'll think about it.

rkorytkowski · 2025-07-28T09:19:01Z

Reverted to use @Autowired in openmrs/openmrs-core@77de212 Once CI build completes, we can rerun tests.

dkayiwa · 2025-07-28T09:22:34Z

@rkorytkowski are we reverting simply for tests? How about making tests hack this out without having to revert the changes in core, if they were for the better running of the application?

wikumChamith · 2025-07-28T09:44:40Z

The issue is TestingApplicationContext overrides sessionFactory from core and doesn't set cacheConfig property. I think I need to move cacheConfig to be autowired again in case of this kind of overwrite... Overwriting bean definition is hacky and we will want to come up with some better approach here. I'll think about it.

Is this a feature or a bug?

rkorytkowski · 2025-07-28T11:19:48Z

My change in Autowired doesn't improve anything in core. It was a leftover after experimenting with code that I didn't think would impact anything... but life can't be that predictable. I reverted the change. Another thing is that overwriting sessionFactory as many modules do in TestingApplicationContext isn't the best approach so we could think of a cleaner idea.

wikumChamith · 2025-07-28T11:39:28Z

My change in Autowired doesn't improve anything in core. It was a leftover after experimenting with code that I didn't think would impact anything... but life can't be that predictable. I reverted the change. Another thing is that overwriting sessionFactory as many modules do in TestingApplicationContext isn't the best approach so we could think of a cleaner idea.

@rkorytkowski, now we are getting a different error.

[INFO] Running org.openmrs.module.emrapi.adt.reporting.evaluator.MostRecentAdmissionRequestVisitDataEvaluatorTest
Warning:  661| ISPN000554: jboss-marshalling is deprecated and planned for removal
Warning:  161| ISPN000554: jboss-marshalling is deprecated and planned for removal
Warning:  863| Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'reportingSerializedDefinitionService' defined in URL [jar:file:/home/runner/.m2/repository/org/openmrs/module/reporting-api/1.25.0/reporting-api-1.25.0.jar!/moduleApplicationContext.xml]: Cannot create inner bean 'org.openmrs.module.reporting.definition.service.SerializedDefinitionServiceImpl#6d709a91' of type [org.openmrs.module.reporting.definition.service.SerializedDefinitionServiceImpl] while setting bean property 'target'; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'org.openmrs.module.reporting.definition.service.SerializedDefinitionServiceImpl#6d709a91' defined in URL [jar:file:/home/runner/.m2/repository/org/openmrs/module/reporting-api/1.25.0/reporting-api-1.25.0.jar!/moduleApplicationContext.xml]: Initialization of bean failed; nested exception is org.spri
java.lang.NullPointerException
	at org.openmrs.api.context.ServiceContext.getRegisteredComponents(ServiceContext.java:921)
	at org.openmrs.api.context.ServiceContext.getRegisteredComponents(ServiceContext.java:889)
	at org.openmrs.api.context.Context.getRegisteredComponents(Context.java:1314)
	at org.openmrs.module.emrapi.encounter.EmrEncounterServiceImpl.onStartup(EmrEncounterServiceImpl.java:99)
	at org.openmrs.module.emrapi.encounter.EmrEncounterServiceImpl$$FastClassBySpringCGLIB$$30c7b921.invoke(<generated>)
	at org.springframework.cglib.proxy.MethodProxy.invoke(MethodProxy.java:218)
	at org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.invokeJoinpoint(CglibAopProxy.java:792)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:163)
	at org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:762)
	at org.springframework.transaction.interceptor.TransactionInterceptor$1.proceedWithInvocation(TransactionInterceptor.java:123)
	at org.springframework.transaction.interceptor.TransactionAspectSupport.invokeWithinTransaction(TransactionAspectSupport.java:388)
	at org.springframework.transaction.interceptor.TransactionInterceptor.invoke(TransactionInterceptor.java:119)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
	at org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:762)
	at org.springframework.aop.framework.CglibAopProxy$DynamicAdvisedInterceptor.intercept(CglibAopProxy.java:707)
	at org.openmrs.module.emrapi.encounter.EmrEncounterServiceImpl$$EnhancerBySpringCGLIB$$fd9f2812.onStartup(<generated>)
	at sun.reflect.GeneratedMethodAccessor91.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:344)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:198)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:163)
	at org.springframework.transaction.interceptor.TransactionInterceptor$1.proceedWithInvocation(TransactionInterceptor.java:123)
	at org.springframework.transaction.interceptor.TransactionAspectSupport.invokeWithinTransaction(TransactionAspectSupport.java:388)
	at org.springframework.transaction.interceptor.TransactionInterceptor.invoke(TransactionInterceptor.java:119)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
	at org.openmrs.aop.LoggingAdvice.invoke(LoggingAdvice.java:123)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
	at org.springframework.aop.framework.adapter.MethodBeforeAdviceInterceptor.invoke(MethodBeforeAdviceInterceptor.java:58)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
	at org.springframework.aop.framework.adapter.MethodBeforeAdviceInterceptor.invoke(MethodBeforeAdviceInterceptor.java:58)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
	at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:241)
	at com.sun.proxy.$Proxy250.onStartup(Unknown Source)
	at org.openmrs.api.context.Daemon.lambda$runInDaemonThreadInternal$5(Daemon.java:452)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:750)

wikumChamith · 2025-07-28T11:46:16Z

My change in Autowired doesn't improve anything in core. It was a leftover after experimenting with code that I didn't think would impact anything... but life can't be that predictable. I reverted the change. Another thing is that overwriting sessionFactory as many modules do in TestingApplicationContext isn't the best approach so we could think of a cleaner idea.

@rkorytkowski, now we are getting a different error.

[INFO] Running org.openmrs.module.emrapi.adt.reporting.evaluator.MostRecentAdmissionRequestVisitDataEvaluatorTest
Warning:  661| ISPN000554: jboss-marshalling is deprecated and planned for removal
Warning:  161| ISPN000554: jboss-marshalling is deprecated and planned for removal
Warning:  863| Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'reportingSerializedDefinitionService' defined in URL [jar:file:/home/runner/.m2/repository/org/openmrs/module/reporting-api/1.25.0/reporting-api-1.25.0.jar!/moduleApplicationContext.xml]: Cannot create inner bean 'org.openmrs.module.reporting.definition.service.SerializedDefinitionServiceImpl#6d709a91' of type [org.openmrs.module.reporting.definition.service.SerializedDefinitionServiceImpl] while setting bean property 'target'; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'org.openmrs.module.reporting.definition.service.SerializedDefinitionServiceImpl#6d709a91' defined in URL [jar:file:/home/runner/.m2/repository/org/openmrs/module/reporting-api/1.25.0/reporting-api-1.25.0.jar!/moduleApplicationContext.xml]: Initialization of bean failed; nested exception is org.spri
java.lang.NullPointerException
	at org.openmrs.api.context.ServiceContext.getRegisteredComponents(ServiceContext.java:921)
	at org.openmrs.api.context.ServiceContext.getRegisteredComponents(ServiceContext.java:889)
	at org.openmrs.api.context.Context.getRegisteredComponents(Context.java:1314)
	at org.openmrs.module.emrapi.encounter.EmrEncounterServiceImpl.onStartup(EmrEncounterServiceImpl.java:99)
	at org.openmrs.module.emrapi.encounter.EmrEncounterServiceImpl$$FastClassBySpringCGLIB$$30c7b921.invoke(<generated>)
	at org.springframework.cglib.proxy.MethodProxy.invoke(MethodProxy.java:218)
	at org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.invokeJoinpoint(CglibAopProxy.java:792)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:163)
	at org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:762)
	at org.springframework.transaction.interceptor.TransactionInterceptor$1.proceedWithInvocation(TransactionInterceptor.java:123)
	at org.springframework.transaction.interceptor.TransactionAspectSupport.invokeWithinTransaction(TransactionAspectSupport.java:388)
	at org.springframework.transaction.interceptor.TransactionInterceptor.invoke(TransactionInterceptor.java:119)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
	at org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:762)
	at org.springframework.aop.framework.CglibAopProxy$DynamicAdvisedInterceptor.intercept(CglibAopProxy.java:707)
	at org.openmrs.module.emrapi.encounter.EmrEncounterServiceImpl$$EnhancerBySpringCGLIB$$fd9f2812.onStartup(<generated>)
	at sun.reflect.GeneratedMethodAccessor91.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:344)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:198)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:163)
	at org.springframework.transaction.interceptor.TransactionInterceptor$1.proceedWithInvocation(TransactionInterceptor.java:123)
	at org.springframework.transaction.interceptor.TransactionAspectSupport.invokeWithinTransaction(TransactionAspectSupport.java:388)
	at org.springframework.transaction.interceptor.TransactionInterceptor.invoke(TransactionInterceptor.java:119)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
	at org.openmrs.aop.LoggingAdvice.invoke(LoggingAdvice.java:123)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
	at org.springframework.aop.framework.adapter.MethodBeforeAdviceInterceptor.invoke(MethodBeforeAdviceInterceptor.java:58)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
	at org.springframework.aop.framework.adapter.MethodBeforeAdviceInterceptor.invoke(MethodBeforeAdviceInterceptor.java:58)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
	at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:241)
	at com.sun.proxy.$Proxy250.onStartup(Unknown Source)
	at org.openmrs.api.context.Daemon.lambda$runInDaemonThreadInternal$5(Daemon.java:452)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:750)

I missed this: openmrs/openmrs-module-reporting@ac35bc7

dkayiwa · 2025-07-30T21:43:21Z

api/src/main/java/org/openmrs/module/emrapi/utils/GeneralUtils.java

        for (int i = 0; i < len; i++) {
            char c = localeString.charAt(i);
            // allow only ASCII letters and "_" character
-            if ((c <= 0x20 || c >= 0x7f) || ((c >= 0x20 || c <= 0x7f) && (!Character.isLetter(c) && c != 0x5f))) {


Is there a reason why you dropped ((c >= 0x20 || c <= 0x7f)?

It is always true in here.

dkayiwa · 2025-07-30T21:44:07Z

Does this compile on Java 21?

dkayiwa · 2025-07-30T21:47:34Z

api/src/main/java/org/openmrs/module/emrapi/patient/HibernateEmrPatientDAO.java

+				.distinct(true);
+
+		TypedQuery<Patient> typed = em.createQuery(cq);
+		if (start != null)     typed.setFirstResult(start);


https://openmrs.atlassian.net/wiki/spaces/docs/pages/25477044/Java+Conventions#Code-Style

dkayiwa · 2025-07-30T21:48:31Z

api/src/main/java/org/openmrs/module/emrapi/patient/HibernateEmrPatientDAO.java

+
 	}

-	private Criteria buildCriteria(String query, Criteria criteria) {


Why did we get rid of this one?

PatientSearchCriteria is deprecated. The same logic is now inside the findPatients method.

if (StringUtils.isNotBlank(query)) { if (query.matches(".*\\d.*")) { Join<Patient, PatientIdentifier> ids = patient.join("identifiers", JoinType.LEFT); preds.add(cb.like(cb.lower(ids.get("identifier")), "%" + query.toLowerCase() + "%")); } else { Join<Patient, PersonName> names = patient.join("names"); String like = "%" + query.toLowerCase() + "%"; preds.add(cb.or(cb.like(cb.lower(names.get("givenName")), like), cb.like(cb.lower(names.get("familyName")), like))); } }

wikumChamith · 2025-07-31T06:38:46Z

Does this compile on Java 21?

No. There are some modules that this depends on that do not yet support Java 21.

dkayiwa · 2025-07-31T10:18:50Z

Can't we do as for other modules that we have given support for all the way from Java 8 to 21?

dkayiwa · 2025-08-11T11:53:39Z

api/src/main/java/org/openmrs/module/emrapi/utils/MetadataUtil.java

@@ -1,341 +0,0 @@
-/**
- * The contents of this file are subject to the OpenMRS Public License


@wikumChamith why did we delete this file?

org.openmrs.module.ModuleException: Unable to start OpenMRS. Error thrown was: org/openmrs/module/emrapi/utils/MetadataUtil

Since this is deprecated and this class also caused an error.

org.openmrs.module.ModuleException: Unable to start OpenMRS. Error thrown was: org/openmrs/module/emrapi/utils/MetadataUtil

In which module?

When starting O2

wikumChamith marked this pull request as draft July 16, 2025 14:57

wikumChamith marked this pull request as ready for review July 22, 2025 05:26

wikumChamith requested review from dkayiwa, ibacher and mseaton July 22, 2025 05:45

mseaton reviewed Jul 22, 2025

View reviewed changes

wikumChamith added 12 commits July 24, 2025 09:11

Fix syntax errors

33c587c

Fix test errors

c9f4dd8

Update to Platform 2.8.0-beta

acbf515

Fix test error

38df407

Fix test error

956ebac

Remove ProviderServiceFacade.java

3198216

Remove EmrApiContextSensitiveTest.java

c74c582

Avoid the use of lenient

cbd6fa6

Add the surefire plugin to the omod pom

bc634b3

Change to Platform 2.8.0-SNAPSHOT

02aeb2d

Remove MetadataUtil.java

c06193f

Set the version to 3.0,0-SNAPSHOT

f0ca4d5

wikumChamith force-pushed the 2.8 branch from 901932e to f0ca4d5 Compare July 24, 2025 03:42

wikumChamith requested a review from mseaton July 24, 2025 06:28

Remove Provider module from aware modules list

e855887

mseaton approved these changes Jul 24, 2025

View reviewed changes

Remove unused imports from HibernateEmrPatientDAO

dcccaee

wikumChamith added 2 commits July 28, 2025 17:17

Update the reporting and calculation modules to the latest snapshot

d831c17

Fix code issues

769b2dd

dkayiwa reviewed Jul 30, 2025

View reviewed changes

Fix code issues

c050545

Fix issues with Java 21

7d459fa

dkayiwa merged commit 0e7f899 into openmrs:master Aug 10, 2025
1 check passed

dkayiwa reviewed Aug 11, 2025

View reviewed changes


		}

		private Criteria buildCriteria(String query, Criteria criteria) {

		@@ -1,341 +0,0 @@
		/**
		* The contents of this file are subject to the OpenMRS Public License

EA-219: Update to OpenMRS Platform 2.8.x #263

EA-219: Update to OpenMRS Platform 2.8.x #263

Uh oh!

Conversation

wikumChamith commented Jul 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

wikumChamith commented Jul 22, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

wikumChamith commented Jul 23, 2025

Uh oh!

wikumChamith commented Jul 23, 2025

Uh oh!

mseaton commented Jul 23, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mseaton left a comment

Choose a reason for hiding this comment

Uh oh!

wikumChamith commented Jul 27, 2025

Uh oh!

dkayiwa commented Jul 27, 2025

Uh oh!

wikumChamith commented Jul 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rkorytkowski commented Jul 28, 2025

Uh oh!

rkorytkowski commented Jul 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

dkayiwa commented Jul 28, 2025

Uh oh!

wikumChamith commented Jul 28, 2025

Uh oh!

rkorytkowski commented Jul 28, 2025

Uh oh!

wikumChamith commented Jul 28, 2025

Uh oh!

wikumChamith commented Jul 28, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

dkayiwa commented Jul 30, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

wikumChamith commented Jul 31, 2025

Uh oh!

dkayiwa commented Jul 31, 2025

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

wikumChamith commented Jul 16, 2025 •

edited

Loading

mseaton commented Jul 23, 2025 •

edited

Loading

wikumChamith commented Jul 28, 2025 •

edited

Loading

rkorytkowski commented Jul 28, 2025 •

edited

Loading