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

chore(advertising): update copies #8562

Merged
merged 6 commits into from
Apr 3, 2023
Merged

chore(advertising): update copies #8562

merged 6 commits into from
Apr 3, 2023

Conversation

caugner
Copy link
Contributor

@caugner caugner commented Apr 3, 2023

Summary

Updates the copy texts of the advertising pages now that the experiment has been finished.

Changes

  1. Removed the "Experimenting with advertising on MDN" page.
  2. Updated the "Advertise with us" page.
  3. Moved the "Advertise with us" page to /advertising.
  4. Added a redirect from /advertising/with_us to /advertising.

Screenshots

Path Before After
/advertising image image
/advertising/with_us image (Removed.)

How did you test this change?

Ran yarn dev locally and visited both http://localhost:3000/advertising and http://localhost:3000/advertising/with_us.

@caugner caugner requested a review from fiji-flo April 3, 2023 13:34
Copy link
Contributor

@fiji-flo fiji-flo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add and redirect and fix all links (rg advertising should help). Otherwise great!

@caugner
Copy link
Contributor Author

caugner commented Apr 3, 2023

Please add and redirect

Added in 8933d6c.

fix all links (rg advertising should help).

Which links do you refer to, afaik there was only one link to the "Advertise with us" page:

image

Copy link
Contributor Author

@caugner caugner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fiji-flo Maybe keep the referrer for Mozilla-internal links?

client/src/advertising/with_us/index.tsx Show resolved Hide resolved
client/src/advertising/with_us/index.tsx Show resolved Hide resolved
client/src/advertising/with_us/index.tsx Show resolved Hide resolved
@caugner caugner force-pushed the MP-317-update-ad-copies branch from a9a825c to 8933d6c Compare April 3, 2023 14:19
@caugner caugner merged commit 18c9231 into main Apr 3, 2023
@caugner caugner deleted the MP-317-update-ad-copies branch April 3, 2023 15:03
wbamberg added a commit to wbamberg/yari that referenced this pull request Apr 4, 2023
* upstream/main: (251 commits)
  chore(deps-dev): bump typescript from 4.9.5 to 5.0.3 in /client/pwa (mdn#8547)
  chore(deps-dev): bump typescript from 4.9.5 to 5.0.3 (mdn#8543)
  chore(deps-dev): bump postcss-preset-env from 8.2.0 to 8.3.0 (mdn#8561)
  chore(deps-dev): bump @types/react from 18.0.31 to 18.0.32 (mdn#8560)
  chore(advertising): update copies (mdn#8562)
  fix(client): place hash before headings (mdn#8532)
  chore(deps-dev): bump ts-jest from 29.0.5 to 29.1.0 (mdn#8559)
  chore(deps-dev): bump black from 23.1.0 to 23.3.0 in /deployer (mdn#8555)
  chore(deps-dev): bump webpack-dev-server from 4.13.1 to 4.13.2 (mdn#8558)
  chore(deps-dev): bump swr from 2.1.1 to 2.1.2 (mdn#8556)
  chore(deps): bump boto3 from 1.26.99 to 1.26.104 in /deployer (mdn#8557)
  chore(deps-dev): bump black in /testing/integration (mdn#8554)
  ci(stage-build): rsync with -j html,json,txt (mdn#8549)
  ci(stage-build): rsync with -c (mdn#8548)
  chore(deps-dev): bump react-router and react-router-dom from 6.9.0 to 6.10.0 (mdn#8534)
  chore(deps-dev): bump @babel/preset-env from 7.20.2 to 7.21.4 (mdn#8544)
  chore(deps-dev): bump tailwindcss from 3.3.0 to 3.3.1 (mdn#8546)
  chore(deps): bump @mdn/browser-compat-data from 5.2.46 to 5.2.47 (mdn#8545)
  chore(deps-dev): bump @babel/core from 7.21.3 to 7.21.4 (mdn#8542)
  fix(docs): make icons in good/bad examples float (mdn#8531)
  ...
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