uPortal-Project / NotificationPortlet

JSR-286 portlet implementing notifications in uPortal.
https://wiki.jasig.org/pages/viewpage.action?pageId=47875986
Apache License 2.0
3 stars 29 forks source link

regeneratorRuntime is not defined #116

Closed jonathanmtran closed 6 years ago

jonathanmtran commented 6 years ago

Describe the bug A regeneratorRuntime is not defined is preventing the modal notification from rendering

To Reproduce

  1. Deploy portlet with a modal notification web component such as jasig/uPortal-start#226
  2. Open developer tools
  3. Navigate to the page containing the modal notification web component

Expected behavior A notification modal

Screenshots screenshot from 2018-10-04 14-33-13

screenshot from 2018-10-04 14-32-43

Platform:

ChristianMurphy commented 6 years ago

ChristianMurphy commented 6 years ago

Looking into this now :mag:

ChristianMurphy commented 6 years ago

@jonathanmtran a work around is adding <script src="https://unpkg.com/regenerator-runtime@0.12.1/runtime.js"></script> https://github.com/uPortal-contrib/uPortal-web-components/issues/117 will probably be the long term fix to prevent this from coming up again as new components get added.

jonathanmtran commented 6 years ago

Adding <script src="https://unpkg.com/regenerator-runtime@0.12.1/runtime.js"></script> works for Chrome/ium On the other hand, the modal doesn't render on Firefox

ChristianMurphy commented 6 years ago

What version of notification are you using?

That should have been fixed when https://github.com/Jasig/NotificationPortlet/blob/e673a002469ecb3179afcf7eb3692814520e555d/notification-portlet-webcomponents/notification-modal/src/components/NotificationModal.vue#L179-L183

was added in https://github.com/Jasig/NotificationPortlet/commit/dcd0f11c88e70e2b360cec62615d115e07329388, included in v4.3.1+

ChristianMurphy commented 6 years ago

Is it still happening in 4.3.1 on your system?

jonathanmtran commented 6 years ago

@ChristianMurphy Yes. I've deployed 4.3.1 and it still doesn't render/appear on Firefox

ChristianMurphy commented 6 years ago

@jonathanmtran I tested this with uPortal 5.3.0 and a snapshot of the master branch of this repo (soon to be officially released). I'm seeing it work on Chrome, Firefox, and Edge.

Let me know if you run into any other issues, we could hop on a video call and debug.

ChristianMurphy commented 6 years ago

Notification portlet 4.4.0 includes the fixes