[SAK-35117] Add Samigo events constants to api to avoid duplication Created: 17-Nov-2015 Updated: 17-Apr-2018 Resolved: 31-Mar-2017
|Component/s:||Tests & Quizzes (Samigo)|
|Fix Version/s:||11.5 [Tentative], 12.0|
|Reporter:||Matthew Jones||Assignee:||SAMIGO TEAM (Inactive)|
|Remaining Estimate:||Not Specified|
|Time Spent:||Not Specified|
|Original Estimate:||Not Specified|
There was some changes added in
+ public static final String EVENT_PUBASSESSMENT_SETTINGS = "sam.pubsetting.edit";
There are probably more of these.
|Comment by Matthew Jones [ 30-Mar-2017 ]|
This may not be a clean merge back to 11 as events may have only been added for 12, but I think it should get back to make merging of other dependent issues possible.
|Comment by Neal Caidin [ 13-Apr-2017 ]|
This sounds like a nice cleanup, but is there any way for QA to verify?
|Comment by Matthew Jones [ 13-Apr-2017 ]|
Probably not without an exhaustive test to verify every event generates correctly. This just might be something we need to merge as it could block merging other issues. The code has been reviewed and it essentially was a search and replace.
|Comment by Neal Caidin [ 25-Apr-2017 ]|
Set to won't fix for now. May need to merge later if there are dependencies that need this.