TOLOGIX - ISLG App Rebuild

Change requirements for Expiry Date for accounts in groups

Assigned to
Harsh Parikh, Tech Lead at DevIT Harsh P. Naomi Joanis, UX Team Lead at Industrial Naomi J.
Notes
For the the Expiry Dates for individual user accounts, after discussions with Liam Murphy-Burke, Strategic Account Manager at Tologix Liam and Marysia Raptis Marysia , we decided that an ideal solution would be to change the requirements so that when an individual user account gets assigned to a group, it would lose its Expiry Date, effectively inheriting the expiry date of the group. This would ensure individual user account expiry dates always match the group expiry dates. Therefore, Harsh Parikh, Tech Lead at DevIT Harsh the following changes are needed:
  1.  When a user account gets assigned to a group:
    • any date entered into the Expiry Date field is deleted, and
    • the admin user cannot edit the field while the user account is assigned to a group.
  2. In addition, the Group Management table currently excludes a column listing the expiry date, we'll want to modify the table an insert a "Expiry" columns between "Type of Account" and "Region":

Comments & Events

Harsh Parikh, Tech Lead at DevIT
Hi Morgan Maguire, CEO Morgan ,

That was my mistake, today I have checked and found that we have set status "Non-Active" when expiry date passed.

The Service is running every night 11:45 PM server time to update the Subscriber and Group status if they passed expiry date.

Today, I have removed that service on app.islg so now if the expiry date is reached then status will be not changed on app.islg.

Hope this is fine.
Morgan Maguire, CEO
Hi Harsh Parikh, Tech Lead at DevIT Harsh ​,

We'd rather have the accounts change to non-active status when the expiry date is reached, so please change it back. Let's ensure it matches the requirements as written in the applicable user stories (at both the user and group level). Also, please confirm my second question about the timing.

Thanks,

Morgan
Harsh Parikh, Tech Lead at DevIT
Hi Morgan Maguire, CEO Morgan ,

The timing is 11:45 PM Server time every day. It means every day the service is running and fetch the subscriber who passed expiry date.

Can we get back change ?  If I get back change then service will run tonight 11:45 PM and all the expiry date groups and subscribers will become 
non-active.

Please confirm.
Morgan Maguire, CEO
Yes,  please Harsh Parikh, Tech Lead at DevIT Harsh ​, but we'll during the call this morning.

Thanks,

Morgan
Morgan Maguire, CEO
Hi Harsh Parikh, Tech Lead at DevIT Harsh ,

I have modified the notes above to clarify that this to-do will now focus only modifying the requirements concerning the Expiry Date for user accounts vis-a-vis groups.

  1. Naomi Joanis, UX Team Lead at Industrial Naomi has created a user to first issue above: https://industrialagency.tpondemand.com/restui/board.aspx?#page=userstory/23284
  2. For the second request on the changes to the group table, I believe sufficient detail is provided in the notes above.
Thanks,

Morgan
Melissa Cowell, General Manager at Industrial
Morgan Maguire, CEO Morgan  

Here is an update on the group expiration requirements based on a call with Harsh & Team this morning:

  1. Currently Harsh has stopped the job that will change subscriber accounts to not active when the group expiration date has passed. That is likely why it seems to be working as you would like right now. He has provided you with a list of the 60-ish users who were affected before he stopped the job.

    Please let Harsh know if you would like this job to be started again. Based on his review 500+ subscribers will be moved to not-active when he does this.

  2. In the current implementation (once the job is running again), when a group reaches the expiry date all subscribers will be changed to not active. If the group status is restored or the expiration date is updated to a date in the future all group users will become active.

    Naomi and I will confirm the original requirements that were not able to be included for launch and Harsh will investigate how to ensure:
  • Expired group users are blocked from accessing the application
  • Individual Subscriber status is maintained from previous when the group is restored. 

Also note, as mentioned on the related Target Process card, in the current implementation, subscriber expiration is already being inherited from the group and is displayed as read-only for group subscribers. If a user is removed from a group, the group expiry date will be removed and then can be managed for the subscriber separately. 

The only instance when an individual group subscriber's expiry date can be managed independently is when there is no expiry date set for the group. Would you like us to remove this functionality?

Let me know if you'd like to discuss further. 

Mel
Morgan Maguire, CEO
Hi Melissa Cowell, General Manager at Industrial Melissa ​,

Yes, I'm aware of the above, but I'd like these requirements changed as explained through my other posts. We don't want want the group expiry date to affect the "status" of the individual user accounts, we only want it to block access when the group account status changes to non-active. As explained in my other posts, the system is currently doing this, we just need to remove the data from expiry date field when the user account is part of a group. This will ensure the user account status is not affected when the group status changes while the group status controls access.

Could you please update the relevant card in Target Process and we'll get this implemented according to the to-do priority list.

Re the SQL job, we'll keep that inactive until we clean up and remove the data from the expiry date field from all the user accounts included in a group and have updated the group expiry date field. We'll need to do this through a back process where we'll provide Harsh Parikh, Tech Lead at DevIT Harsh ​with updated field data in spreadsheets and then he'll need to update the relevant accounts in the database.

Thanks,

Morgan

BTW, regarding the 6am, Monday development call. Could you please ensure Martin Laporte, CTO at Tologix Martin ​is included in this call going forward.

Thanks,

Morgan

 
Morgan Maguire, CEO
Hi Harsh Parikh, Tech Lead at DevIT Harsh and Melissa Cowell, General Manager at Industrial Melissa

Please implement the user stories above according the the priorities of this to-do. Note that the only effective change to the current requirements is:
  1. If a user account is part of a group, delete the data in the Expire Date field for the user account.
  2. If a user account is part of a group, an admin user will not be able to edit the Expiry Date field (which will be blank) when editing the details for a user.
According to my tests performed on Friday, all other requirements concerning how group status affects user account access are already integrated into the requirements.

Please implement on staging.islg, and then after testing we'll get this integrated into app.islg.

Thanks,

Morgan
Naomi Joanis, UX Team Lead at Industrial
Morgan Maguire, CEO Morgan User stories updated to reflect the discussion today. We can discuss with Harsh Parikh, Tech Lead at DevIT Harsh  on Monday as well to confirm the understanding.
Morgan Maguire, CEO
Perfect. Thanks Naomi Joanis, UX Team Lead at Industrial Naomi .
Harsh Parikh, Tech Lead at DevIT
Hi Morgan Maguire, CEO Morgan and Naomi Joanis, UX Team Lead at Industrial Naomi ,

Today, We have deployed new version on both app.islg and staging.islg. so this task has been deployed on both app.islg and staging.islg.

Now, Subscriber will be not able to login/auto-login, if expiry date is passed. Could you please check and let us know the feedback.

I have also moved both stories in UAT.
Morgan Maguire, CEO
Hi Harsh Parikh, Tech Lead at DevIT Harsh ​,

It concerns me a little that we deployed the change directly on app.islg without fully testing this change on staging first, but I'll setup some tests today to make sure this is working correctly.

Naomi Joanis, UX Team Lead at Industrial Naomi ​, could you please perform UAT on the applicable user stories and let us know if you find any problems we will need to deal with. I'm conscious of what impacts any issues will have on active subscribers.

Thanks,

Morgan
Harsh Parikh, Tech Lead at DevIT
Hi Morgan Maguire, CEO Morgan ,

Due to Document Comparison task, We need to also publish this task on app.islg because today we need to deploy new version.

Due to this reason, We have published this task directly on app.islg.

Our QC is checked fully. But, you can also test and check and if there is any issue then will resolve quickly.
Naomi Joanis, UX Team Lead at Industrial
Hi Morgan Maguire, CEO Morgan

I've set my test group's expiry for tomorrow so I will verify both user stories completely by then. I did notice however that in the group > edit page, there is a message that states "Status of all group's users will change to Non-Active upon reaching expiry date." which is not according to the criteria. I will verify if this is how it is working by tomorrow. 

The story regarding users inheriting the group's expiry (https://industrialagency.tpondemand.com/entity/23317-islg-team-feedback-group-user-expiry) does seem to be working properly, however. 

Thanks, 

Naomi
Harsh Parikh, Tech Lead at DevIT
Hi Naomi Joanis, UX Team Lead at Industrial Naomi ,

"Status of all group's users will change to Non-Active upon reaching expiry date."  This is just old message which we removed now on both app.islg and staging.islg.

Please check again.
Morgan Maguire, CEO
Great. Thanks Naomi Joanis, UX Team Lead at Industrial Naomi ​.

I've setup my own test as well. I also noticed that user accounts assigned to groups still have their expiry dates. My understanding was that these would be eliminated when the user is assigned to a group, but I'll let this get addressed as part of UAT.

Harsh Parikh, Tech Lead at DevIT Harsh ​, we'll have to closely monitor this and make sure we don't disrupt access to users on app.islg.

Also, Liam Murphy-Burke, Strategic Account Manager at Tologix Liam ​and Marysia Raptis Marysia ​, we'll have to make it a priority task today to update all expiry date fields for Subscriber Groups, because there is a strong possibility the expiry dates at the individual user account level will get erased by tonight when the system updates.

Thanks,

Morgan    
Naomi Joanis, UX Team Lead at Industrial
 Hi Morgan Maguire, CEO Morgan and Harsh Parikh, Tech Lead at DevIT Harsh

Testing the expired accounts story this morning, both auto-IP access and login was blocked for me which was good. However the following issues seem to be occurring with the criteria:
  • "All associated user detail pages will display an alert to admins noting that the group expiration date has been reached, the date of expiration."
    • The date of expiration on the message is missing
  • "All user details except the Expiry Date field can still be edited and saved by admins"
    • I am not able to edit the "Status" field
  • "When a group's expiration date is met, the user's status will not change but their access to the application will be blocked"
    • The user status was changed to non-active
Morgan Maguire, CEO
Hi Naomi Joanis, UX Team Lead at Industrial Naomi ​,

Could you provide details on what accounts you used for testing purposes along with screenshots, because my tests produced conflicting results from above.

Thanks,

Morgan
Morgan Maguire, CEO
Hi Naomi Joanis, UX Team Lead at Industrial Naomi ,

Following-up on above, further to the video below, the only outstanding issues I've found on staging.islg is that the status of the group is not changing to Non-Active when the expiration date is reached. Otherwise, everything is working accordingly to the requirements. Could you please clarify if your getting conflicting results.

Thanks,

Morgan

Naomi Joanis, UX Team Lead at Industrial
Hi Morgan Maguire, CEO Morgan

Please see screenshots/videos for the issues I mentioned above. As well I tested this on staging with Industrial Agency test as the group and my user that is the only member of the group which is "Naomi Test":

Morgan Maguire, CEO
Ok. That makes sense Naomi Joanis, UX Team Lead at Industrial Naomi ​.

Harsh Parikh, Tech Lead at DevIT Harsh ​, it looks like there are four issues to resolve:
  • The status of the group is not changing to Non-Active when the Expiry Date of the group is reached.
  • The Expiry Date for the group is missing from the message displayed with edit mode for individual user accounts.
  • The status of the individual user account is blocked from editing when the Expiry Date for the group is reached.
  • When the Expiry Date for the group is reached the status of the individual user account is changed to Non-Active when you view the account on the edit page (even though the status displays a different status in the user table). The status for the individual user account should not be affected by the group expiry date (it should only block access).

Could we please resolve this on staging.islg and then assuming everything is resolved, we'll deploy the changes to app.islg during the next deployment window at on Monday at 11:30pm EST (Tuesday at 9am IST).

Liam Murphy-Burke, Strategic Account Manager at Tologix Liam ​ and Marysia Raptis Marysia ​, please not that currently individual user account access is blocked (both auto-IP and user/pass) when the group expiration date is reached, even when the group's status is Active.

Thanks,

Morgan
Morgan Maguire, CEO
Hi Harsh Parikh, Tech Lead at DevIT Harsh ,

Further to the video below, I found another issue with individual user accounts that are not assigned to a group. According to the requirements, if a individual user account is not assigned to a group and date in the Expiry Date is reached, it should change the status of the account to Non-Active. That is not occurring currently, the status on the account is not changing when the Expiry Date is reach. A message is displayed indicating the "User expiration date has been reached", but it's unclear if this is blocking access to the platform.



Thanks,

Morgan

Harsh Parikh, Tech Lead at DevIT
Hi Morgan Maguire, CEO Morgan ,

We are working on the above issues and it will be completed by tomorrow on staging.islg. so will deploy this task on app.islg by 11th May.
Harsh Parikh, Tech Lead at DevIT
Hi Morgan Maguire, CEO Morgan and Naomi Joanis, UX Team Lead at Industrial Naomi ,

The above all issues have been done on staging.islg. You can check by adding new group and user.

Please check and confirm.
Morgan Maguire, CEO
Hi Harsh Parikh, Tech Lead at DevIT Harsh ,

I'll setup some tests on staging and let you know the results. However, I'm going to perform the tests on existing accounts, because we need to ensure this work on existing accounts.

Thanks,

Morgan
Morgan Maguire, CEO
Hi Harsh Parikh, Tech Lead at DevIT Harsh ​,

I just checked my test and the following issue is still unresolved:

  • The status of the group is not changing to Non-Active when the Expiry Date of the group is reached.

Thanks,

Morgan
Harsh Parikh, Tech Lead at DevIT
Hi Morgan Maguire, CEO Morgan ,

We found bug while JOB is running at 11:45 PM on Staging database to make group Non-Active. We resolved it and we put one more group for testing purpose.

We will let you know by tomorrow once our test will be done on staging.islg.
Naomi Joanis, UX Team Lead at Industrial
Hi Morgan Maguire, CEO Morgan

Confirming that my tests concluded the same as you, the outstanding issue I see is the status of the expired group not changing. 

Thanks, 

Naomi
Morgan Maguire, CEO
Hi Harsh Parikh, Tech Lead at DevIT Harsh ,

Looks like the bug isn't resolved. Further to the screenshot below, the group I have setup for testing purposes (Investor-State LawGuide (Seasons))
didn't change to "Non-Active" status at 11:45pm EST.


Thanks,

Morgan 
Morgan Maguire, CEO
Hi Harsh Parikh, Tech Lead at DevIT Harsh and Naomi Joanis, UX Team Lead at Industrial Naomi ,

Following-up on above, it appears that was just a delay in the SQL job from executing and changing the status of my test account, the status of the account has changed to "Non-Active":
However, I've found two other issues that need to be examined:
  1. Assigning a user account to a group changes the status of the account to "Non-Active" and the admin user does not have the ability to edit the status of the account: 
  2. When the Expiry Date for a user account that is not assigned to a group is reached, access to the system is blocked, but the status of the account does not change to "Non Active":
For issue #2, I'm open to feedback on whether we should leave things as is and allow the status to go unchanged when the Expiry Date is reached, but it seems inconsistent with how we're dealing with the issue at the group level. Therefore, we should probably make it so that the status changes to "Non Active" when the Expiry Date is reached. Naomi Joanis, UX Team Lead at Industrial Naomi , do you agree?

Thanks,

Morgan
Harsh Parikh, Tech Lead at DevIT
Hi Morgan Maguire, CEO Morgan ,

The above all issues have been resolved on staging.islg. Please check and confirm.

For #2nd issue in above comment, Please Provide feedback.
Naomi Joanis, UX Team Lead at Industrial
Hi Morgan Maguire, CEO Morgan

I do agree this would be a more consistent approach, my only concern would be that the user's status prior to the expiry date would get lost. If it's possible that changing the expiration date of the user can restore the user status this would be the best approach. However, these were the original requirements and I know there were some technical issues in implementing this. 

Please let me know your thoughts here if you think that would create an issue for your team. 

Thanks, 

Naomi
Morgan Maguire, CEO
Hi Harsh Parikh, Tech Lead at DevIT Harsh and Naomi Joanis, UX Team Lead at Industrial Naomi ,

After talking through the issue with Liam Murphy-Burke, Strategic Account Manager at Tologix Liam we've decided not to implement issue #2 above. When the Expiry Date is reached for individual user accounts, access to the system should be blocked (i.e., the user credentials will no longer work on the Subscriber site), but the status of the account should not change.

I believe all the requirements for this to-do are now resolved on staging.islg and I'll mark the to-do complete.

Harsh Parikh, Tech Lead at DevIT Harsh , please migrate these changes to app.islg during the next deployment window.

Thanks,

Morgan 
Morgan Maguire, CEO
Morgan Maguire completed this to-do.
Morgan Maguire, CEO
Hi Liam Murphy-Burke, Strategic Account Manager at Tologix Liam and Marysia Raptis Marysia ,

Note that the changes above are now deployed on app.islg. Please note that the following group have expired or will be expiring this week and that action will need to be taken to ensure access is not blocked when the expiration date is reached.


Thanks,

Morgan
Morgan Maguire, CEO
Morgan Maguire re-opened this to-do.
Morgan Maguire, CEO
Hi Harsh Parikh, Tech Lead at DevIT Harsh ,

As a final step to complete this to-do, we need to delete existing data within the Expiry Date field for all user accounts that are currently assigned to a group. Could you please proceed with deleting such data from staging.islg. Once we confirm this was performed successfully on staging.islg, we will do the same on app.islg.

Further to the screenshot below, this will apply to accounts that are assigned to a group and because the expiry date was assigned before we implemented the changes above, the date still appears in the details for the user account:


Thanks,

Morgan
Harsh Parikh, Tech Lead at DevIT
Hi Morgan Maguire, CEO Morgan ,

We have removed Expiry Date form users who are associated with group on staging.islg. Please check and verify.

If it will be OK then will do same for app.islg.
Morgan Maguire, CEO
Hi Harsh Parikh, Tech Lead at DevIT Harsh ,

Everything looks good. Please proceed with making the same on app.islg.

Liam Murphy-Burke, Strategic Account Manager at Tologix Liam and Marysia Raptis Marysia , please note that note that when this change is implemented as user accounts assigned to group will have the date in their Expiry Date field deleted. Therefore, please ensure any user accounts that need expiration dates tracked at the user account level (e.g., trial accounts) are removed from groups before EOD today.

Thanks,

Morgan
Harsh Parikh, Tech Lead at DevIT
Hi Morgan Maguire, CEO Morgan ,

We did this change on app.islg through query.  so the expiry date is blank for the users who are associated with group on app.islg.

Please check and confrim.
Morgan Maguire, CEO
Hi Harsh Parikh, Tech Lead at DevIT Harsh ,

Everything looks good.

Marysia Raptis Marysia and Liam Murphy-Burke, Strategic Account Manager at Tologix Liam , note that all the user accounts assigned to a group no longer have a date in the expiry field.

Marking the to-do complete.

Morgan
Morgan Maguire, CEO
Morgan Maguire completed this to-do.