Fwd: FW: Remaining work this week for Demand Management

1 view
Skip to first unread message

Subodh Khare

unread,
May 13, 2013, 7:08:48 PM5/13/13
to pasdm
Hi Navneet,

Stacy is having issues with the code on logging and e-mail alerts, do you what is the reason ? The details are given below.

I have setup a meeting tomorrow morning 8:30 am (PST) with Stacy to go over these issues. Can you join this meeting and clarify any issues he has reported. I would also like to have minutes from these meetings so that, Stacy does not keep reporting all these issues.
(I have another meeting during this time so will not be able to join)

Team, rest of the review comments will discuss in tomorrow's meeting


Regards

Subodh


---------- Forwarded message ----------
From: Khare, Subodh <Subodh...@goaaa.com>
Date: Mon, May 13, 2013 at 4:03 PM
Subject: FW: Remaining work this week for Demand Management
To: "subodh...@saama.com" <subodh...@saama.com>


 

 

From: O'Dell, Stacy
Sent: Monday, May 13, 2013 3:34 PM
To: Khare, Subodh
Cc: Rivard, Fiona
Subject: Remaining work this week for Demand Management
Importance: High

 

Please consider these items in roughly their order of importance.  If you have any questions, please reach out to me immediately.

 

Regards,

 

Stacy O’Dell

Technical Delivery Specialist V – Insurance Applications

 

AAA Northern California, Nevada & Utah Insurance Exchange

5353 W. Bell Road  Glendale, AZ 85308

p: 602.588.5043   c: 623.850.1007

stacy...@goaaa.com

 


 

RTC Integration

·         Need export of their configuration from their environment.  A list of custom fields is insufficient.

·         Need to test integration on our environment (NOTE from Stacy:  Once I receive the proper configuration information for RTC, I will set it up and test the integration here.  I just want to make sure we are able to test it before the week is out.  The sooner the better)

 

Authentication Need to implement Windows Authentication. 

·         I think the login.aspx and logout.aspx pages can go away if properly implemented.

·         Use HttpContext.Current.User to get currently logged-in user (NOTE:  HttpContext.Current.User is a System.Security.Principal.Iprincipal object, so you can write the code referencing that interface and mock the values in your environment if you like, then we can set it to HttpContext.Current.User in the future)

·         If a user exists in the database, update.  If not, add

o    Pull additional information from AD (at least stub this code out so we can complete it)

·         Delete Role table and any related stored procedures (and code)

·         Delete UserRole table and any related stored procedures (and code)

·         Create new cross reference table to tie users to Domains (call it "DomainManagers")

·         Modify admin UI for User Management

o    Change title to "Domain Managers"

o    Change "Roles" column to be "Domains"

o    Only list domains for which the user is a manager in the new "Domains" column

·         Display login in "editing users" dialog (ID is insufficient)

·         In "Add User" dialog change "Alias" to "Login"

o    Add validation to make sure Login is a required field

·         (later we will add additional validation to ensure login is a valid user in AD, but it is not necessary for Saama to implement this)

·         Modify code to use HttpContext.Current.User.IsInRole method to test for role membership instead of roles table.

o    Move definition of role names to web.config so they can be substituted later easily

·         SystemAdmin

·         EnterpriseManager

·         DomainManager

 

Logging

·         Logging doesn't appear to be working at all.  Nothing is logged.

 

 

Email Alerts

·         Email alerts doesn't appear to be functional either.  What sends emails?  Are they sent in real time?  Via a background process?  We could find no indication in the code which would cause an email alert to actually be sent.

·         Move configuration to web.config instead of hard coding values

 

General

·         Add the name of the submitter to all tables and dialogs so you can see who submitted it

·         The ability to scroll and view request deatails submitted by others (when in “view” or “read only” mode, you cannot scroll the multiline text box for description, and if there is a large description, you cannot read it all)

·         When you view “all Items”, then click on an item to view it’s details, then hit the “Back” button, it reverts back to “My items”

·         Implement pagination on all tables.  Page size should be configurable (site wide setting via web.config)

·         Need to know exactly what Sean changed in HighCharts -- send us the difference reports between the original and current code

·         Implement UI and database changes necessary to associate a change request with zero or more Clubs (including a selection for “All”)

·         Implement UI and database changes necessary to associate a change request with zero or more States (including a selection for “All”)

·         When prioritizing items as a domain manager, there is no link to view an item’s complete details

·         When prioritizing items as an enterprise manager, there is no link to view an items complete details.

·         Dialogs should be implemented consistently throughout, including

    • close “X” button in upper right hand corner (and eliminate any superfluous “close” buttons”)
    • Same javascript library used for all
    • Same color scheme
    • Dialogs should be centered if possible

 




--
Subodh Khare | Associate Principal| Saama Technologies, Inc. | Office: 408-371-1900 | mobile: 650-619-7758 | www.saama.com

This e-mail transmission may contain confidential, proprietary and / or legally privileged information and is intended only for the individual or entity named in the e-mail address. Any disclosure, copying, distribution, or reliance upon the contents of this e-mail not authorized by the sender is strictly prohibited. If you have received this e-mail transmission in error, please immediately reply to the sender, so that proper delivery of the e-mail can be effected, and then please delete the message from your Inbox. Any content of this message and its attachments that does not relate to the official business of Saama Technologies Inc. or its subsidiaries must be taken not to have been sent or endorsed by any of them. Email communications are not private and no warranty is made that e-mail communications are timely, secure or free from computer virus or other defect.

Navneet Kumar

unread,
May 13, 2013, 10:33:43 PM5/13/13
to Subodh Khare, pasdm

This may be a setup specific issue, Kalpana can you investigative?
I'm in mumbai today and tomorrow (14th and 15th) for my visa interview but ill try to join the meeting from hotel.

Regards
Navneet

> --
> You received this message because you are subscribed to the Google Groups "PAS Demand Management" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to pasdm+un...@googlegroups.com.
> For more options, visit https://groups.google.com/groups/opt_out.
>  
>  

David Lee

unread,
May 14, 2013, 2:56:53 AM5/14/13
to Navneet Kumar, Subodh Khare, pasdm
Yea when we went through the code review we could see the code but it didn't appear to be hooked up for both emails and logging. Kalpana can you make sure this is all working on your end.
thanks
-David

Ashish Chauhan

unread,
May 14, 2013, 3:18:55 PM5/14/13
to Subodh Khare, pasdm
Hi Subodh,

Below are my comments.

Delete Role table and any related stored procedures (and code) - I can implement this change in AAA environment as we don't have AD configuration setup.  After deleting this table, we don't have any way to test different roles for Demand Management System. This change will not take more than few hours.

·         Delete UserRole table and any related stored procedures (and code) -  I can implement this change in AAA environment as we don't have AD configuration setup.  After deleting this table, we don't have any way to test different roles for Demand Management System. This change will not take more than few hours.



Thanks
-Ashish


--
You received this message because you are subscribed to the Google Groups "PAS Demand Management" group.
To unsubscribe from this group and stop receiving emails from it, send an email to pasdm+un...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
 
 
Reply all
Reply to author
Forward
0 new messages