# OpenMRS Platform 2.3.0-beta released for testing **URL:** \ **Category:** Platform\ **Tags:** releases\ **Created:** [September 20, 2019, 1:47pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795 "2019-09-20T13:47:19Z")\ **Posts on this page:** 20\ **Page:** 5
**Author:** ![ruhanga](https://talk.openmrs.org/user_avatar/talk.openmrs.org/ruhanga/32/8318_2.png) [@ruhanga](https://talk.openmrs.org/u/ruhanga)\ **Post date:** [January 4, 2020, 7:39pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/81 "2020-01-04T19:39:27Z")
Thanks @mogoodrich, > [@mogoodrich](#): > > Are Patient/Person the only things using annotations right now? Yes, as far as changes related to the data filter module were made. > [@wyclif](#): > > I will just revert back to xml @wyclif, I hope that we have no interference when these changes(annotations on Person/Patient) are reverted. ---
**Author:** ![dkayiwa](https://talk.openmrs.org/user_avatar/talk.openmrs.org/dkayiwa/32/179_2.png) [@dkayiwa](https://talk.openmrs.org/u/dkayiwa)\ **Post date:** [January 8, 2020, 6:53pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/82 "2020-01-08T18:53:27Z")
@ruhanga any updates on this? ---
**Author:** ![ruhanga](https://talk.openmrs.org/user_avatar/talk.openmrs.org/ruhanga/32/8318_2.png) [@ruhanga](https://talk.openmrs.org/u/ruhanga)\ **Post date:** [January 9, 2020, 12:19pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/83 "2020-01-09T12:19:38Z")
A work around with hibernate mappings on the side of the data filter module is to be made before we can revert the introduced annotations on the Person/Patient domains. cc: @wyclif ---
**Author:** ![wyclif](https://talk.openmrs.org/user_avatar/talk.openmrs.org/wyclif/32/19607_2.png) [@wyclif](https://talk.openmrs.org/u/wyclif)\ **Post date:** [January 9, 2020, 11:12pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/84 "2020-01-09T23:12:05Z")
Why should we revert from annotations for Person/Patient? We reverted for some of the other classes because they required adding dummy tables which we are not doing for Person and Patient @mogoodrich there were issues of bad test data and not behavior changes after switching to annotations. ---
**Author:** ![dkayiwa](https://talk.openmrs.org/user_avatar/talk.openmrs.org/dkayiwa/32/179_2.png) [@dkayiwa](https://talk.openmrs.org/u/dkayiwa)\ **Post date:** [January 9, 2020, 11:25pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/85 "2020-01-09T23:25:13Z")
How about the change of behavior that led to this ticket? [https://issues.openmrs.org/browse/TRUNK-5691](https://issues.openmrs.org/browse/TRUNK-5691) ---
**Author:** ![mogoodrich](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mogoodrich/32/3324_2.png) [@mogoodrich](https://talk.openmrs.org/u/mogoodrich)\ **Post date:** [January 10, 2020, 1:58pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/86 "2020-01-10T13:58:09Z")
Yeah, thanks @dkayiwa that was one issue. It could be argued that this is a small issue that doesn’t manifest itself in “real life” much (though I haven’t done detailed analysis). Bigger picture, though, it would seem best to me to have a consistent way to set up mappings, not annotations for Patient/Person but mapping files for everything else, so if there’s no longer a need to switch Patient/Person I’d strongly recommend reverting for consistency and to avoid the above the issue (and any other issues we may have missed). No point in adding something that could introduce issues if there’s not a compelling need. Take care, Mark ---
**Author:** ![ruhanga](https://talk.openmrs.org/user_avatar/talk.openmrs.org/ruhanga/32/8318_2.png) [@ruhanga](https://talk.openmrs.org/u/ruhanga)\ **Post date:** [January 13, 2020, 11:50am UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/87 "2020-01-13T11:50:34Z")
@wyclif, I hope we can safely go ahead to revert from annotations for Person/Patient if this has no implications to the ongoing developments on the data filter module ? ---
**Author:** ![ruhanga](https://talk.openmrs.org/user_avatar/talk.openmrs.org/ruhanga/32/8318_2.png) [@ruhanga](https://talk.openmrs.org/u/ruhanga)\ **Post date:** [January 13, 2020, 4:02pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/88 "2020-01-13T16:02:59Z")
Hi all, we are about to have the Platform 2.3.0 released by the 21st and from the PM call today, this will pave way for the pending Reference Application release. We hope that by the 17th January (latest) there are no issues pending soon after the requested changes/reverts are effected. Thank you all for your tireless efforts contributing to OpenMRS. 🙂 Regards ---
**Author:** ![ruhanga](https://talk.openmrs.org/user_avatar/talk.openmrs.org/ruhanga/32/8318_2.png) [@ruhanga](https://talk.openmrs.org/u/ruhanga)\ **Post date:** [January 20, 2020, 6:55pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/89 "2020-01-20T18:55:04Z")
Hi all, I’ll be reverting the following commits as a follow up on the changes introduced due to Patient/Person domain objects being annotated. [TRUNK-5492](https://github.com/openmrs/openmrs-core/commit/53776880d02a899cbd93e618a3bab6a037a02dc2) [TRUNK-5685](https://github.com/openmrs/openmrs-core/commit/ebb491d2b9af15421e913a3ee9b12b52f8a06b5e) [TRUNK-5685](https://github.com/openmrs/openmrs-core/commit/1a38e83edfc5ac083b28d6d4f5fa46b9084268e8) [TRUNK-5492](https://github.com/openmrs/openmrs-core/commit/9c3a95fb93f63082711c84ce4724ab30c7b5dab1) [TRUNK-5675](https://github.com/openmrs/openmrs-core/commit/ded28def7940b9b73277025ba270988a2c95c915) The last commit in the above sequence probably was addressing an issue that arose due to the Person/Patient domain objects being annotated. @bistenes, @mogoodrich , would reverting this, affect progress of developments that led to this? Thank you. Kind regards. Nathan ---
**Author:** ![mogoodrich](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mogoodrich/32/3324_2.png) [@mogoodrich](https://talk.openmrs.org/u/mogoodrich)\ **Post date:** [January 21, 2020, 6:23pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/90 "2020-01-21T18:23:09Z")
@ruhanga I don’t think TRUNK-5675 is related to the Person/Patient domain object change and we wouldn’t want to revert it… if I’m missing something and reverting the Person/Patient changes _without_ reverting TRUNK-5675 causes issues, lmk… Also, I don’t know if there’s any harm in keeping TRUNK-5685? Seems nice to keep that protected so subclasses can access it? Thanks! Mark ---
**Author:** ![ruhanga](https://talk.openmrs.org/user_avatar/talk.openmrs.org/ruhanga/32/8318_2.png) [@ruhanga](https://talk.openmrs.org/u/ruhanga)\ **Post date:** [January 21, 2020, 8:48pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/91 "2020-01-21T20:48:33Z")
Thanks @mogoodrich. Yes, reverting the annotation changes on Patient/Person affects the commits on accents (TRUNK-5675), I get failing tests related to the commit on this. Also the reverts on Patient/Person annotations causes failing tests related to TRUNK-5685 for the protected method. A safe revert from annotations on Patient/Person domain object currently requires the above sequence of commits get reverted where all tests pass. Perhaps with a few tweaks we could keep the commit on TRUNK-5675. ---
**Author:** ![mogoodrich](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mogoodrich/32/3324_2.png) [@mogoodrich](https://talk.openmrs.org/u/mogoodrich)\ **Post date:** [January 21, 2020, 9:00pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/92 "2020-01-21T21:00:57Z")
Thanks @ruhanga! Let me take a quick look… ---
**Author:** ![mogoodrich](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mogoodrich/32/3324_2.png) [@mogoodrich](https://talk.openmrs.org/u/mogoodrich)\ **Post date:** [January 21, 2020, 11:03pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/93 "2020-01-21T23:03:10Z")
@ruhanga I was doing some testing with the 2.3.x and couldn’t reproduce the issues you were seeing, but then I just tested with master and was getting some errors but ran out of time tonight, but will need to look more into it… Reverting TRUNK-5685 isn’t a problem, but we will to keep TRUNK-5675… That being said, if you want to go ahead and revert it for now, feel free, just please reopen the ticket and flag me here so I can look into it further. Thanks! Mark ---
**Author:** ![ruhanga](https://talk.openmrs.org/user_avatar/talk.openmrs.org/ruhanga/32/8318_2.png) [@ruhanga](https://talk.openmrs.org/u/ruhanga)\ **Post date:** [January 22, 2020, 9:08pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/94 "2020-01-22T21:08:54Z")
@mogoodrich, after building the branch 2.3.x with the reverted changes except for TRUNK-5675 in a windows and macOS environments, some tests related to TRUNK-5675 failed on windows. There is [a fix](https://github.com/openmrs/openmrs-core/commit/ddd39ac848df067ab5914981e0b541a609af9fb9) that was not backported to the 2.3.x branch. I hope I can release the platform as soon as it is backported. Otherwise there are no other blockers and we can keep TRUNK-5675. Thank you. 🙂 ---
**Author:** ![mogoodrich](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mogoodrich/32/3324_2.png) [@mogoodrich](https://talk.openmrs.org/u/mogoodrich)\ **Post date:** [January 22, 2020, 10:11pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/95 "2020-01-22T22:11:30Z")
Thanks @ruhanga, good catch… I will backport that now… ---
**Author:** ![mogoodrich](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mogoodrich/32/3324_2.png) [@mogoodrich](https://talk.openmrs.org/u/mogoodrich)\ **Post date:** [January 22, 2020, 10:43pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/96 "2020-01-22T22:43:26Z")
@ruhanga backport has been applied, it’s going through the build now, let me know if you notice any issues! ---
**Author:** ![wyclif](https://talk.openmrs.org/user_avatar/talk.openmrs.org/wyclif/32/19607_2.png) [@wyclif](https://talk.openmrs.org/u/wyclif)\ **Post date:** [January 23, 2020, 3:08am UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/97 "2020-01-23T03:08:43Z")
I was going crazy about what annotations have to do with accents in strings. I personally strongly believe that we should use annotations and start to move away from xml because xml masks bugs that would otherwise be exposed by annotations since they are part of the java language, also a lot of advancements and libraries of the language are implemented with annotations. ---
**Author:** ![mozzy](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mozzy/32/11815_2.png) [@mozzy](https://talk.openmrs.org/u/mozzy)\ **Post date:** [January 23, 2020, 6:12am UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/98 "2020-01-23T06:12:50Z")
@wyclif , i personally agree with you. My opinion was not to do reverts but rather to put in more effort to switch completely to annotations ---
**Author:** ![dkayiwa](https://talk.openmrs.org/user_avatar/talk.openmrs.org/dkayiwa/32/179_2.png) [@dkayiwa](https://talk.openmrs.org/u/dkayiwa)\ **Post date:** [January 23, 2020, 8:38am UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/99 "2020-01-23T08:38:56Z")
I had thought that a switch to annotations would be obvious. But it turned out not to be so, basing on the side effects that we have experienced, and probably more would show up. Therefore, instead of annotations delaying the 2.3.0 release, i would bump them to 2.4.0 or even 3.0.0 ---
**Author:** ![ruhanga](https://talk.openmrs.org/user_avatar/talk.openmrs.org/ruhanga/32/8318_2.png) [@ruhanga](https://talk.openmrs.org/u/ruhanga)\ **Post date:** [January 23, 2020, 2:27pm UTC](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795/100 "2020-01-23T14:27:21Z")
Thank you @mogoodrich, @dkayiwa, @wyclif, @mozzy and @all for your thoughts and support. For now I’m going ahead to release the platform without the annotations for the purpose of consistency and not having to postponed this release. However annotations on domain objects are more contemporary and convenient for use. This could be part of our GSoC projects and future releases. [Previous page](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795.md?page=4) [Next page](https://talk.openmrs.org/t/openmrs-platform-2-3-0-beta-released-for-testing/24795.md?page=6)