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

Release Notes JSON Description Field #3829

Merged
merged 3 commits into from Sep 13, 2021
Merged

Release Notes JSON Description Field #3829

merged 3 commits into from Sep 13, 2021

Conversation

cade-conklin
Copy link
Contributor

Give us some context

The current release notes JSON return a JSON with fields: agent, version, and release date. One of our services relies on this JSON but needs a description field so that it can parse the release notes to determine the supportability for various agents. The change was very minor and involved adding an excerpt field to the GraphQL call. The length of each excerpt was arbitrarily set at 5000 assuming most release notes won't exceed that length. When running locally, this is what the new JSON will look like
Screen Shot 2021-09-08 at 1 54 51 PM

Here is a link to the issue that was opened a while ago. The reason this was just now completed was that our team just now raised visibility on it again.

Are you making a change to site code?

Yes. This is a very small change!

If you're changing site code (rather than the content of a doc), please follow
conventional commit standards
in your commit messages and pull request title.

@CLAassistant
Copy link

CLAassistant commented Sep 8, 2021

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added this to Hero to triage in Docs PRs and Issues Sep 8, 2021
@gatsby-cloud
Copy link

gatsby-cloud bot commented Sep 8, 2021

Gatsby Cloud Build Report

docs-website-develop

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 40m

@x8a x8a added content requests related to docs site content from_external Identifies issues/PRs from non-Relics eng issues related to site functionality that requires engineering and removed content requests related to docs site content labels Sep 9, 2021
@x8a
Copy link
Contributor

x8a commented Sep 9, 2021

Hey, @jpvajda! Could you have a look at this? Thanks!

@x8a x8a removed this from Hero to triage in Docs PRs and Issues Sep 9, 2021
@jpvajda
Copy link

jpvajda commented Sep 10, 2021

I'm going to see if @roadlittledawn can review as well. It looks good to me, but he did some work on this part of the site so wanted another set of 👀 on it. Thanks for the contribution @x8a and @cade-conklin

Copy link
Contributor

@aswanson-nr aswanson-nr left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@jpvajda jpvajda merged commit a9e35a6 into newrelic:develop Sep 13, 2021
Developer Enablement Team automation moved this from In review to Done Sep 13, 2021
Developer Enablement Team automation moved this from Done to In review Sep 13, 2021
Copy link
Contributor

@zstix zstix left a comment

Choose a reason for hiding this comment

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

The first time I ran this my environment crashed, but every other time this has worked well, so I'm chalking it up to something on my end 🎉

@aswanson-nr aswanson-nr moved this from In review to Done in Developer Enablement Team Sep 13, 2021
@austin-schaefer austin-schaefer added from_internal Identifies issues/PRs from Relics (except writers) and removed from_external Identifies issues/PRs from non-Relics labels Sep 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
eng issues related to site functionality that requires engineering from_internal Identifies issues/PRs from Relics (except writers)
Development

Successfully merging this pull request may close these issues.

None yet

8 participants