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

[DX-1437] Update api-def-common.md #5089

Merged
merged 1 commit into from
Jul 16, 2024
Merged

[DX-1437] Update api-def-common.md #5089

merged 1 commit into from
Jul 16, 2024

Conversation

letzya
Copy link
Collaborator

@letzya letzya commented Jul 16, 2024

User description

For internal users - Please add a Jira DX PR ticket to the subject!



Preview Link


Description


Screenshots (if appropriate)


Checklist

  • I have added a preview link to the PR description.
  • I have reviewed the suggestions made by our AI (PR Agent) and updated them accordingly (spelling errors, rephrasing, etc.)
  • I have reviewed the guidelines for contributing to this repository.
  • I have read the technical guidelines for contributing to this repository.
  • Make sure you have started your change off our latest master.
  • I labelled the PR

PR Type

documentation


Description

  • Corrected the spelling of "organisation" to "organization" in the slug section of the API definition documentation.

Changes walkthrough 📝

Relevant files
Documentation
api-def-common.md
Correct spelling in API definition documentation                 

tyk-docs/content/shared/api-def-common.md

  • Corrected spelling of "organisation" to "organization".
+1/-1     

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

Copy link
Contributor

PR Reviewer Guide 🔍

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No key issues to review

Copy link
Contributor

PR Code Suggestions ✨

No code suggestions found for PR.

Copy link

netlify bot commented Jul 16, 2024

PS. Pls add /docs/nightly to the end of url

Name Link
🔨 Latest commit ff378ab
🔍 Latest deploy log https://app.netlify.com/sites/tyk-docs/deploys/669614c45112010008c276b1
😎 Deploy Preview https://deploy-preview-5089--tyk-docs.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.

1 similar comment
Copy link

netlify bot commented Jul 16, 2024

PS. Pls add /docs/nightly to the end of url

Name Link
🔨 Latest commit ff378ab
🔍 Latest deploy log https://app.netlify.com/sites/tyk-docs/deploys/669614c45112010008c276b1
😎 Deploy Preview https://deploy-preview-5089--tyk-docs.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.

@letzya letzya disabled auto-merge July 16, 2024 07:39
@letzya letzya merged commit b29c0ce into master Jul 16, 2024
10 checks passed
@letzya letzya deleted the letzya-letz-gb-to-us-org branch July 16, 2024 07:39
@letzya
Copy link
Collaborator Author

letzya commented Jul 16, 2024

/release to release-5.4

Copy link

tykbot bot commented Jul 16, 2024

Working on it! Note that it can take a few minutes.

tykbot bot pushed a commit that referenced this pull request Jul 16, 2024
Update api-def-common.md

(cherry picked from commit b29c0ce)
Copy link

tykbot bot commented Jul 16, 2024

@letzya Succesfully merged PR

buger added a commit that referenced this pull request Jul 16, 2024
[DX-1437] Update api-def-common.md (#5089)

Update api-def-common.md
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.

1 participant