ubuntu.com icon indicating copy to clipboard operation
ubuntu.com copied to clipboard

Update Pro beta links to point to /pro/beta

Open sowasred2012 opened this issue 2 years ago • 4 comments

Done

  • Updated links on /pro to point to /pro/beta rather than the Discourse post
  • appled p-button to a "Contact us" link

QA

  • Visit https://ubuntu-com-12208.demos.haus/pro
  • See that the "Contact us" link looks like a button
  • Find the "Discover the step-by-step tutorial" link and click it
  • See that you are taken to /pro/beta
  • Revisit /pro
  • Find the "Tutorials" link and click it
  • See that you are taken to /pro/beta

sowasred2012 avatar Oct 19 '22 14:10 sowasred2012

Demo starting at https://ubuntu-com-12208.demos.haus

webteam-app avatar Oct 19 '22 14:10 webteam-app

Codecov Report

Merging #12208 (e9ebf6f) into main (25e14ce) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main   #12208   +/-   ##
=======================================
  Coverage   63.50%   63.50%           
=======================================
  Files         117      117           
  Lines        2691     2691           
  Branches      791      791           
=======================================
  Hits         1709     1709           
  Misses        940      940           
  Partials       42       42           
Impacted Files Coverage Δ
...ns/SubscriptionDetails/DetailsTabs/DetailsTabs.tsx 92.77% <ø> (ø)

codecov[bot] avatar Oct 19 '22 14:10 codecov[bot]

@sowasred2012 can you please do a drive by? add this to the css: text-decoration-thickness: 1px; text-underline-offset: 0.075em; image

lyubomir-popov avatar Oct 19 '22 15:10 lyubomir-popov

@lyubomir-popov sure - updated and referenced the issue in our CSS, could you leave an approving review if you're happy with this.

sowasred2012 avatar Oct 19 '22 23:10 sowasred2012