Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
yingbull pushed 4 commits to develop/bullfrog cc-ar-emr/Open-O
- update: upgraded JDK version from 11 to 17 - Updated JDK version number. - Removed `java.security.acl.Group` package... e7a9b06
- fix: added opens to bypass the security restrictions in Java 17 318aa55
- update: upgraded JDK version from 17 to 21 434c5f0
- Merge pull request #45 from cc-ar-emr/java17-upg cb220f3
yingbull closed a pull request on cc-ar-emr/Open-O
Java 21 Upgrade
- Upgraded to Java 21 - Resolved the removed package issues - Added opens to resolve the access issue ## Summary by Sourcery Upgrade the project to Java 21, refactor OscarGroup to implement Pri...
yingbull pushed 2 commits to develop/bullfrog cc-ar-emr/Open-O
yingbull pushed 2 commits to develop/bullfrog cc-ar-emr/Open-O
yingbull closed a pull request on cc-ar-emr/Open-O
Add help command to `make` file
## Changes made - Add help command to `make` file. - Display help output if no argument provided.
yingbull pushed 2 commits to develop/bullfrog cc-ar-emr/Open-O
yingbull closed a pull request on cc-ar-emr/Open-O
Set up `over_ride_config.properties` in `make` script
## Changes made - Update `make` to set up `over_ride_config.properties` in the `build_and_deploy()` function.
yingbull created a review comment on a pull request on cc-ar-emr/Open-O
at best, make an issue ticket for this for further investigation and share with magenta. Good to update the docs but if it said it should work a particular way that is likely important. keep the ...
yingbull created a review comment on a pull request on cc-ar-emr/Open-O
sourcery don't change your behaviour. This was important. Please be sure to comment on these issues.
yingbull created a review comment on a pull request on cc-ar-emr/Open-O
ok, but is it right? at worst, an issue ticket should track this finding if you feel it has merit, even if it is just put in the category for pre-existing.
yingbull created a review comment on a pull request on cc-ar-emr/Open-O
ok, if it is pre existing and must be used we can keep it.
yingbull pushed 29 commits to develop/bullfrog cc-ar-emr/Open-O
- feat:update struts a512f9a
- update struts.xml ec12f1f
- refactor: replace <html> taglib with standard html - replace <html:html> with <html> - replace <html:button> with <... 6224271
- fix: change the position of struts,xml fc3d33e
- update: remove alll codes related to struts 1 76e220a
- refactor: replace <bean> tags with JSTL equivalents in jsps - replace <bean:message> tag with <fmt:message> 6e106ae
- refactor: replace <bean> tags with JSTL equivalents in jsps 2 - replace <bean:write> tags with <c:out> - replace ... af690eb
- fix:adjust the position of struts 2 filter 0e90460
- refactor: replace <html:base> with <base> tag 8c5986b
- refactor: replace html taglib 37c741f
- fix: remove struts 1 code in jsp 6613b13
- fix: remove all struts 1 code in JSP 590a848
- fix: remove all struts 1 code in JSP 6076c7f
- refactor: update struts fe2d9b0
- refactor: update struts 1 - replace html taglib with jstl - rewrite Action in struts 2 formation refactor: replac... 14605fc
- refactor: fix action path in JSP 3c637d3
- refactor: fix action path in JSP 62449d7
- refactor: fix action path in JSP 6985178
- fix: fix some bug cause by struts upgrade 3dc51d9
- fix: set action response to method called 38bcde9
- and 9 more ...
yingbull closed a pull request on cc-ar-emr/Open-O
Issue 181 update struts 1
Remove struts 1 and use struts 2 instead. - remove all struts 1 Action class, use OldName2Action as the struts2 action className, if the old name doesn't end with Action, then append 2Action di...
yingbull created a review comment on a pull request on cc-ar-emr/Open-O
lots done here - I assume with respect to quartz compatibility in new version. need to explain that in a commit. ideally its own for it, but it should be stated more specifically what's going on ...
yingbull created a review comment on a pull request on cc-ar-emr/Open-O
best practice is to avoid wildcard imports - can we just list the needed ones?
yingbull created a review comment on a pull request on cc-ar-emr/Open-O
why the O word? can it be OpenoGroup?
yingbull created a review comment on a pull request on cc-ar-emr/Open-O
what is your response, Kate? anything from sorcery should have a response before I get to the PR.