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

Remove edge server mentions #3388

Merged
merged 1 commit into from
Jan 24, 2025
Merged

Conversation

MongoCaleb
Copy link
Collaborator

@MongoCaleb MongoCaleb commented Jan 23, 2025

Pull Request Info - SDK Docs Consolidation

Jira ticket: DOCSP-46788

Staging Links

  • sdk/dotnet/app-services/connect-to-app-services-backend
  • sdk/kotlin/app-services/connect-to-app-services-backend
  • sdk/node/app-services/connect-to-app-services-backend
  • sdk/react-native/app-services/connect-to-app-services-app
  • sdk/swift/app-services/connect-to-app-services-backend
  • Page Source

    Add links to every SDK's pages where you got the SDK-specific information:

    Release Notes

    PR Author Checklist

    Before requesting a review for your PR, please check these items:

    • Open the PR against the feature-consolidated-sdk-docs branch instead of master
    • Tag the consolidated page for:
      • genre
      • meta.keywords
      • meta.description

    Naming

    Links and Refs

    • Create new consolidated SDK ref targets starting with "_sdks-" for relevant sections
    • Remove or update any SDK-specific refs to use the new consolidated SDK ref targets
    • Update any Kotlin API links to use the new Kotlin SDK roles

    Content

    • Shared code boxes have snippets or placeholders for all 9 languages
    • API description sections have API details or a generic placeholder for all 9 languages
    • Check related pages for relevant content to include
    • Create a ticket for missing examples in each relevant SDK: Consolidation Gaps epic

    Reviewer Checklist

    As a reviewer, please check these items:

    • Shared code example boxes contain language-specific snippets or placeholders for every language
    • API reference details contain working API reference links or generic content
    • Realm naming/language has been updated
    • All relevant content from individual SDK pages is present on the consolidated page

    Copy link

    netlify bot commented Jan 23, 2025

    Deploy Preview for device-sdk ready!

    Name Link
    🔨 Latest commit 0f6adc5
    🔍 Latest deploy log https://app.netlify.com/sites/device-sdk/deploys/6792b5a96a4b8f000845bb19
    😎 Deploy Preview https://deploy-preview-3388--device-sdk.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 site configuration.

    Copy link
    Collaborator

    @krollins-mdb krollins-mdb left a comment

    Choose a reason for hiding this comment

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

    Thanks for cleaning up Edge Server content in the SDK docs!

    @MongoCaleb MongoCaleb merged commit 956552b into mongodb:master Jan 24, 2025
    9 of 11 checks passed
    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