Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MWPW-108710 - Handle Emdashes #7

Merged
merged 1 commit into from
Apr 22, 2022
Merged

MWPW-108710 - Handle Emdashes #7

merged 1 commit into from
Apr 22, 2022

Conversation

auniverseaway
Copy link
Member

  • Change makeRelative to swap emdashes for double hyphen.

Resolves: MWPW-108710

Test URLs:

* Change makeRelative to swap emdashes for double hyphen.

Resolves: MWPW-108710
@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Apr 22, 2022

Page Score PSI Audit Google
/fr/ SI FCP LCP TBT CLS PSI

@codecov
Copy link

codecov bot commented Apr 22, 2022

Codecov Report

Merging #7 (98b92dc) into main (bb4a222) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
+ Coverage   94.00%   94.01%   +0.01%     
==========================================
  Files           6        6              
  Lines         350      351       +1     
==========================================
+ Hits          329      330       +1     
  Misses         21       21              
Impacted Files Coverage Δ
scripts/scripts.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2993a1f...98b92dc. Read the comment docs.

@auniverseaway auniverseaway merged commit cf43ae6 into main Apr 22, 2022
@chrischrischris chrischrischris deleted the handle-emdash branch October 12, 2022 19:43
Axelcureno added a commit that referenced this pull request Apr 17, 2024
milo-pr-merge bot pushed a commit that referenced this pull request Dec 4, 2024
#3258)

* move common snippets to helper, add defer logic and make interact call the 1st thing in the flow for signed out users

* toggle FPID and ECID logic, re-organise exports, dont fire if consent cookie present

* Pre merge branch (#5)

move methods utils , use async load of helper, review comments

* move try catch block as per promise

* Pre merge branch (#6)

* Test remove martech logic (#7)

fix delay logic, fix pr comments

* Test remove martech logic (#8)

change uuid logic, promise structure, alloy response logic, requestid,

* Final local branch (#9)

fix review comments, add test cases

* fix coverage for if check

* fix coverage errors

* add test case for martech helpers

* send full cookie flag added post testing round 1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants