CDR Tickets

BZDATETIME::2012-05-30 12:42:32
BZCOMMENTOR::Volker Englisch
BZCOMMENT::1

The vendor filter has been modified and this is ready for review on MAHLER:
CDR000157 - Vendor Filter: Summary

BZDATETIME::2012-05-30 14:36:00
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::2

(In reply to comment #1)
> The vendor filter has been modified and this is ready for review on MAHLER:
> CDR000157 - Vendor Filter: Summary

What will be a good way to test this? Should we wait until some of the summaries are ready and run them through publishing?

BZDATETIME::2012-05-30 15:00:30
BZCOMMENTOR::Volker Englisch
BZCOMMENT::3

You would need to look at the XML vendor output to ensure the attribute has been set properly since we don't have a publish preview for mobile yet.

BZDATETIME::2012-05-30 15:28:11
BZCOMMENTOR::Margaret Beckwith
BZCOMMENT::4

We also need to get the mobile urls for these first, don't we?

BZDATETIME::2012-05-30 15:40:36
BZCOMMENTOR::Volker Englisch
BZCOMMENT::5

Yes, we still need to load the Mobile URLs but loading the URLs to the document and testing the vendor filter changes are two separate tasks. We just can't run publishing without both being completed.

BZDATETIME::2012-05-30 15:42:28
BZCOMMENTOR::Volker Englisch
BZCOMMENT::6

There was an error sending out the email. Entering this comment to trigger it.

BZDATETIME::2012-05-31 11:57:43
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::7

So in preparing a Screening/Prevention summary for testing, I am making up the mobile URL to follow the existing patter http://m.cancer.gov....... (Is this necessary for testing?)
Should I also be assigning the Exclude attributes in the 4 sections or you handle that in the Vendor filters?
After preparing the document, I am using the Filter Document Interface to review the XML.

BZDATETIME::2012-06-05 16:49:40
BZCOMMENTOR::Margaret Beckwith
BZCOMMENT::8

Lisa will be providing the correct mobile urls for the screening and prev. summaries. I will ask her where that stands so that you can use a real url for testing.

BZDATETIME::2012-06-12 14:22:27
BZCOMMENTOR::Volker Englisch
BZCOMMENT::9

I don't think we had discussed this so I want to make sure the questions are answered:

(In reply to comment #7)
> So in preparing a Screening/Prevention summary for testing, I am making up the
> mobile URL to follow the existing patter http://m.cancer.gov....... (Is this
> necessary for testing?)

No, the mobile URL is not necessary for testing here. It is needed when processing the documents on Gatekeeper.
The test for this filter change, however, is to confirm that the "ExcludedDevice='Mobile'" has been added properly by the filter to those sections indicated above on all screening and prevention summaries.

> Should I also be assigning the Exclude attributes in the 4 sections or you
> handle that in the Vendor filters?

The vendor filter is adding this attribute but you might want to test that an attribute specified by the user will not be overwritten by the vendor filter.

> After preparing the document, I am using the Filter Document Interface to
> review the XML.

Correct. You would have to look for this change by looking at the XML output.

BZDATETIME::2012-06-14 11:47:25
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::10

(In reply to comment #9)

> > Should I also be assigning the Exclude attributes in the 4 sections or you
> > handle that in the Vendor filters?
>
> The vendor filter is adding this attribute but you might want to test that an
> attribute specified by the user will not be overwritten by the vendor filter.
>
> > After preparing the document, I am using the Filter Document Interface to
> > review the XML.
>
> Correct. You would have to look for this change by looking at the XML output.

I did look at some patient screening and prevention summaries and all of the attributes displayed correctly in those summaries.

BZDATETIME::2012-06-20 14:19:04
BZCOMMENTOR::Margaret Beckwith
BZCOMMENT::11

This is the spreadsheet for the mobile urls for PDQ patient screening and prevention summaries that Lisa sent me. It is possible that these may change, but since I am going to be away for a few days, I wanted these to be available for testing in the event that they don't change. If they do change, I will ask Lisa to send Volker the new spreadsheet.

Either CIAT or Volker will need to put a few of these into summaries on Mahler for testing, but we don't want to insert them in the summaries on Bach until after Release 6.5 has gone live. At that point, CIAT can insert them into all of the summaries on Bach.

Attachment PreventionScreeningSummaryURLs.xls has been added with description: Mobile urls for screening and prevention summaries

BZDATETIME::2012-06-20 21:34:22
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::12

(In reply to comment #11)
> Either CIAT or Volker will need to put a few of these into summaries on Mahler
> for testing, but we don't want to insert them in the summaries on Bach until
> after Release 6.5 has gone live. At that point, CIAT can insert them into all
> of the summaries on Bach.

CIAT will add the URLs to a few summaries on Mahler and let Volker know.

BZDATETIME::2012-06-21 15:05:48
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::13

(In reply to comment #12)

> CIAT will add the URLs to a few summaries on Mahler and let Volker know.

The following summaries have been updated with the URLs and made publishable for testing:

454520 Cancer Prevention Overview;Prevention;Patients
062822 Ovarian Cancer Prevention;Prevention;Patients
062802 Skin Cancer Prevention;Prevention;Patients

304478 Bladder and Other Urothelial Cancers Screening;Screening;Patients
258010 Endometrial Cancer Screening;Screening;Patients
315400 Stomach (Gastric) Cancer Screening;Screening;Patients

All of the above summaries contain images except for Cancer Prevention Overview.
Bladder has table images + mobile images, with the include/exclude

BZDATETIME::2012-06-28 14:10:04
BZCOMMENTOR::Margaret Beckwith
BZCOMMENT::14

Adding spreadsheet with new urls for the screening and prevention summaries. William, please correct the urls in the test documents so that Volker can republish them when he gets back.

Attachment PreventionScreeningSummaryURLs.xls has been added with description: Mobile urls for screening and prevention patient summaries

BZDATETIME::2012-06-29 12:34:03
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::15

(In reply to comment #14)

> William, please correct the urls in the test documents so that Volker can
> republish them when he gets back.

The following summaries have been corrected and are ready to be republished:

CDR304478 Bladder and Other Urothelial Cancers Screening
CDR258010 Endometrial Cancer Screening
CDR315400 Stomach (Gastric) Cancer Screening

BZDATETIME::2012-07-06 18:53:50
BZCOMMENTOR::Volker Englisch
BZCOMMENT::16

(In reply to comment #14)
> William, please correct the urls in the test documents so that Volker can
> republish them when he gets back.

I have republished the three summaries that William updated but these summaries failed to load on Gatekeeper - probably because of the change to the mobilURL.

BZDATETIME::2012-07-23 14:36:30
BZCOMMENTOR::Volker Englisch
BZCOMMENT::17

I don't see a comment that indicates that the test documents had been tested/reviewed on the mobile test site.

I'm assuming that it's OK to move the filter changes to production now for the screening/prevention summaries to go to mobile.
Is that correct?

BZDATETIME::2012-07-23 14:41:40
BZCOMMENTOR::Margaret Beckwith
BZCOMMENT::18

Yes, it can go to production. I asked CIAT to put all of the mobile urls in on Bach.

BZDATETIME::2012-07-23 17:00:53
BZCOMMENTOR::Volker Englisch
BZCOMMENT::19

The following filter has been copied to BACH:
CDR000157: Vendor Filter: Summary - R10470

The HP disclaimer text (OCECDR-3489) can now be previewed in PublishPreview.
For the module and deep-linking changes the documents will need to be prepared and for the screening and prevention changes the MobileURL element will need to be populated. The ExcludeDevices attribute could be checked in the XML.

BZDATETIME::2012-09-07 11:06:41
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::20

We decided to close this issue in yesterday's CDR status meeting.

Marking as Resolved - fixed.

BZDATETIME::2012-09-07 11:06:56
BZCOMMENTOR::William Osei-Poku
BZCOMMENT::21

Issue closed.