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

Added more examples to Response Helpers page #29462

Merged
merged 28 commits into from
Oct 13, 2021

Conversation

molebox
Copy link
Contributor

@molebox molebox commented Sep 28, 2021

Bug

Feature

  • Implements an existing feature request or RFC. Make sure the feature request has been accepted for implementation before opening a PR.
  • Related issues linked using fixes #number
  • Integration tests added
  • Documentation added
  • Telemetry added. In case of a feature if it's used or not.
  • Errors have helpful link attached, see contributing.md

Documentation / Examples

  • Make sure the linting passes

Copy link
Member

@styfle styfle left a comment

Choose a reason for hiding this comment

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

Did you intend to add a bunch of .tgz files?

@ijjk

This comment has been minimized.

molebox and others added 5 commits September 28, 2021 18:18
Co-authored-by: Steven <steven@ceriously.com>
Co-authored-by: Steven <steven@ceriously.com>
Co-authored-by: Steven <steven@ceriously.com>
Co-authored-by: Steven <steven@ceriously.com>
Co-authored-by: Steven <steven@ceriously.com>
@molebox
Copy link
Contributor Author

molebox commented Sep 28, 2021

Did you intend to add a bunch of .tgz files?

I didnt! I installed the dpes and they appeared. I wasnt sure what the deal was as its the first time ive run the project. Whats the play here?

@ijjk

This comment has been minimized.

@ijjk

This comment has been minimized.

@ijjk

This comment has been minimized.

@molebox
Copy link
Contributor Author

molebox commented Oct 6, 2021

This should be good to go now @timneutkens

molebox and others added 5 commits October 13, 2021 09:13
Co-authored-by: JJ Kasper <jj@jjsweb.site>
Co-authored-by: JJ Kasper <jj@jjsweb.site>
Co-authored-by: JJ Kasper <jj@jjsweb.site>
Co-authored-by: JJ Kasper <jj@jjsweb.site>
@molebox molebox requested a review from ijjk October 13, 2021 07:13
@molebox molebox requested a review from ijjk October 13, 2021 14:28
@kodiakhq kodiakhq bot merged commit eed4311 into canary Oct 13, 2021
@kodiakhq kodiakhq bot deleted the refactor-reponse-helpers-page branch October 13, 2021 14:58
The included helpers are:

- `res.status(code)` - A function to set the status code. `code` must be a valid [HTTP status code](https://en.wikipedia.org/wiki/List_of_HTTP_status_codes)
- `res.json(body)` - Sends a JSON response. `body` must be a [serialiazable object](https://developer.mozilla.org/en-US/docs/Glossary/Serialization)
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi - wanted to point out that serializable is spelled wrong here (it's wrong in current production as well)

Copy link
Member

Choose a reason for hiding this comment

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

Good catch, opened PR here updating this #29899

@ijjk ijjk mentioned this pull request Oct 14, 2021
1 task
kodiakhq bot pushed a commit that referenced this pull request Oct 14, 2021
Fixes typo per #29462 (comment)

## Documentation / Examples

- [x] Make sure the linting passes
@vercel vercel locked as resolved and limited conversation to collaborators Jan 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants