search

NCIOCPL / clinical-trials-search-print

Print service for clinical trial search
0 stars 1 forks source link

Missing CSS and JS assets on legacy print pages #72

Closed blairlearn closed 2 years ago

blairlearn commented 2 years ago

Issue description

When print pages generated by the legacy system are displayed on the lower tiers, they reference CSS and JavaScript files which are not available. These files will be available on production (served from Akamai NetStorage), but in the meantime, this interferes with testing.

ESTIMATE TBD

Steps to reproduce the issue

  1. Find a print ID value which predates the new CTS.Print (e.g. on Dev, DEA341F9-FF2C-EC11-82F6-0050568E76B3)
  2. Open the F12 tools.
  3. Go to the print display path (e.g. https://www-blue-dev.cancer.gov/CTS.Print/Display?printID=DEA341F9-FF2C-EC11-82F6-0050568E76B3)

What's the expected result?

What's the actual result?

Additional details / screenshot

Related Tickets

alinai commented 2 years ago

Not seeing 404 status for all CSS and JavaScript anymore. All CSS and JavaScript returns status 200.

But @blairlearn , @sarinapadilla , getting the error "Uncaught (in promise) ReferenceError: attachEvents is not defined" on refreshing the Print page:

Step to reproduce:

  1. Go to https://ncigovcdode450.prod.acquia-sites.com/about-cancer/treatment/clinical-trials/search
  2. Perform Search
  3. Select a trial and open the F12 tools and open the console.
  4. Print
  5. The print page loads with no error on the console
  6. Refresh the page.
  7. The error "Uncaught (in promise) ReferenceError: attachEvents is not defined" appears. image
sarinapadilla commented 2 years ago

@alinai that error is not something we are going to address for launch. I'll create a ticket for investigation later.