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: added the team field to transform node.js #600

Conversation

rcmtcristian
Copy link
Contributor

@rcmtcristian rcmtcristian commented Sep 6, 2022

Resolves #582

@rcmtcristian rcmtcristian linked an issue Sep 6, 2022 that may be closed by this pull request
@ghost
Copy link

ghost commented Sep 6, 2022

👇 Click on the image for a new way to code review
  • Make big changes easier — review code in small groups of related files

  • Know where to start — see the whole change at a glance

  • Take a code tour — explore the change with an interactive tour

  • Make comments and review — all fully sync’ed with github

    Try it now!

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

@jtfairbank jtfairbank self-requested a review September 6, 2022 16:52
@@ -1,7 +1,7 @@
/*
Page Tree Traversal Helpers
------------------------------------------------------------
NOTE: In JS so it can be used by `/gatsby-node.js` too.
NOTE: In JS so it can be used by `/.js` too.
Copy link
Contributor

Choose a reason for hiding this comment

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

Please undo this unrelated change! Otherwise it all looks good!

Copy link
Contributor

@jtfairbank jtfairbank left a comment

Choose a reason for hiding this comment

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

See comment requesting changes.

@rcmtcristian rcmtcristian temporarily deployed to preview September 6, 2022 16:59 Inactive
@jtfairbank jtfairbank merged commit 46fffbe into saga Sep 6, 2022
@jtfairbank jtfairbank deleted the 582-feat-update-role-data-model-in-gatsby-by-loading-additional-field branch September 6, 2022 17:10
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.

Feat: Update role data model in Gatsby by loading additional field
2 participants