Skip to content

DOCSP-52058 CRUD app with Rocket title #194

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

Merged
merged 1 commit into from
Jul 22, 2025

Conversation

mballard-mdb
Copy link
Contributor

@mballard-mdb mballard-mdb commented Jul 22, 2025

Pull Request Info

PR Reviewing Guidelines

JIRA - https://jira.mongodb.org/browse/DOCSP-52058

Staging Links

  • fundamentals/crud/web-app-tutorial
  • Self-Review Checklist

    • Is this free of any warnings or errors in the RST?
    • Did you run a spell-check?
    • Did you run a grammar-check?
    • Are all the links working?
    • Are the facets and meta keywords accurate?
    • Are the page titles greater than 20 characters long and SEO relevant?

    Copy link

    netlify bot commented Jul 22, 2025

    Deploy Preview for docs-rust ready!

    Name Link
    🔨 Latest commit 7a4178a
    🔍 Latest deploy log https://app.netlify.com/projects/docs-rust/deploys/687f9e1929e3980008e85332
    😎 Deploy Preview https://deploy-preview-194--docs-rust.netlify.app
    📱 Preview on mobile
    Toggle QR Code...

    QR Code

    Use your smartphone camera to open QR code link.

    To edit notification comments on pull requests, go to your Netlify project configuration.

    @docs-builder-bot
    Copy link

    🔄 Deploy Preview for docs-rust processing

    Item Details
    🔨 Latest Commit dc6e4b7fb9a4b1ef1d5900d158e138d7b8cd8e62
    😎 Deploy Preview https://deploy-preview-194--docs-rust.netlify.app
    🔍 Build Logs View Logs

    Copy link
    Collaborator

    @mongoKart mongoKart left a comment

    Choose a reason for hiding this comment

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

    LGTM w/ 1 change

    Tutorial: Create a CRUD Web App with Rocket
    ===========================================
    ========================
    CRUD Web App with Rocket
    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    I think we should keep tutorial: at the beginning, like we've done for other tutorials adapted from dev center articles

    Suggested change
    CRUD Web App with Rocket
    Tutorial: CRUD Web App with Rocket

    Copy link
    Contributor Author

    Choose a reason for hiding this comment

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

    "Titles should be between 30-60 characters." Adding Tutorial would violate these boundaries and negate the purpose of cutting the title down. I wonder if adding "Tutorial:" is necessary to convey the purpose of the page?

    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    I'm seeing 34 characters for the title I suggested...?

    Copy link
    Contributor Author

    Choose a reason for hiding this comment

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

    The full title includes "Tutorial: CRUD Web App with Rocket - Rust Driver v3.2 - MongoDB Docs" according to the JIRA ticket, which would make it 68 characters.

    @mballard-mdb mballard-mdb merged commit ca80ca6 into mongodb:master Jul 22, 2025
    7 checks passed
    github-actions bot pushed a commit that referenced this pull request Jul 22, 2025
    @mballard-mdb mballard-mdb deleted the DOCSP-52058-title-length branch July 22, 2025 17:47
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    3 participants