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

feat: update page titles #3623

Merged
merged 6 commits into from
Jun 13, 2024
Merged

Conversation

benfurber
Copy link
Member

@benfurber benfurber commented Jun 5, 2024

PR Checklist

PR Type

  • New feature (non-breaking change which adds functionality)

Description

By not doing this before we've probably been missing out on traffic...

From --> To
Research --> Research - [Community Name]
[Research Title] --> [Research Title] - Research - [Community Name]
How-tos --> How-tos - [Community Name]
[How-to Title] --> [How-to Title] - How-to - [Community Name]
Questions --> Questions - [Community Name]
[Question Title] --> [Question Title] - Question - [Community Name]
Profile --> [Username] - Profile - [Community Name]

Plus this sets social tags of questions correctly.

Git Issues

Closes #3409

Copy link

cypress bot commented Jun 5, 2024

2 flaky tests on run #5797 ↗︎

0 75 1 0 Flakiness 2

Details:

test: change login command to skip ui by default
Project: onearmy-community-platform Commit: 31c078c3c6
Status: Passed Duration: 04:25 💡
Started: Jun 13, 2024 4:38 PM Ended: Jun 13, 2024 4:42 PM
Flakiness  research/write.spec.ts • 1 flaky test • ci-chrome

View Output Video

Test Artifacts
[Research] > [Create research article] > [By Authenticated] Test Replay Screenshots Video
Flakiness  howto/read.spec.ts • 1 flaky test • ci-chrome

View Output Video

Test Artifacts
... > [See all info] Test Replay Screenshots Video

Review all test suite changes for PR #3623 ↗︎

@benfurber benfurber force-pushed the feat/page-titles branch 3 times, most recently from 86c3b6b to 08fbb0c Compare June 6, 2024 14:32
Copy link

codecov bot commented Jun 6, 2024

Codecov Report

Attention: Patch coverage is 92.30769% with 1 line in your changes missing coverage. Please review.

Project coverage is 68.41%. Comparing base (df53a15) to head (08fbb0c).

Current head 08fbb0c differs from pull request most recent head 064c8bf

Please upload reports for the commit 064c8bf to get more accurate results.

Files Patch % Lines
src/pages/Question/QuestionPage.tsx 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3623      +/-   ##
==========================================
- Coverage   74.87%   68.41%   -6.47%     
==========================================
  Files          39      449     +410     
  Lines        1214    14012   +12798     
  Branches      247     2502    +2255     
==========================================
+ Hits          909     9586    +8677     
- Misses        285     4384    +4099     
- Partials       20       42      +22     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@benfurber
Copy link
Member Author

@davehakkens I don't think this is a controversial change but can you eyeball it please and shout if you have any questions. I strongly argue for it with a product manager hat on but obvs don't want you in the dark.

@benfurber benfurber force-pushed the feat/page-titles branch 2 times, most recently from 064c8bf to 53ed80d Compare June 13, 2024 15:31
@benfurber benfurber merged commit 8a877cd into ONEARMY:master Jun 13, 2024
19 checks passed
@onearmy-bot
Copy link
Collaborator

🎉 This PR is included in version 1.190.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Questions: Page title should update with question title
3 participants