Skip to content

Resolve some tech debt in the prettyTime module

username-removed-408230 requested to merge 37298-the-prettytime-utils-module-could-use-some-light-maintenance into master

What does this MR do?

  1. remove prettyTime from global (gl) scope
  2. export its functions individually
  3. Don't init from in IIFE
  4. Adds a utility function to DRY up the spec

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

  • Tests added for this feature/bug
  • Review
    • Has been reviewed by Frontend

What are the relevant issue numbers?

Closes #37298 (closed)

Edited by username-removed-502136

Merge request reports