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

advise(react/errorboundary): add & change example case #221

Merged

Conversation

minsoo-web
Copy link
Member

Overview

This PR is for clear example case for #216

PR Checklist

  • I have written documents and tests, if needed.

@minsoo-web minsoo-web self-assigned this Oct 15, 2023
@changeset-bot
Copy link

changeset-bot bot commented Oct 15, 2023

⚠️ No Changeset found

Latest commit: d698532

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Oct 15, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
visualization ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 15, 2023 11:43pm
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) Visit Preview Oct 15, 2023 11:43pm

@codecov
Copy link

codecov bot commented Oct 15, 2023

Codecov Report

Merging #221 (d698532) into react/fix/useErrorBoundary-to-stable (9d739d6) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@                          Coverage Diff                           @@
##           react/fix/useErrorBoundary-to-stable      #221   +/-   ##
======================================================================
  Coverage                                100.00%   100.00%           
======================================================================
  Files                                        21        21           
  Lines                                       842       842           
  Branches                                    149       149           
======================================================================
  Hits                                        842       842           
Components Coverage Δ
@suspensive/react 100.00% <ø> (ø)
@suspensive/react-query ∅ <ø> (∅)

@vercel vercel bot temporarily deployed to Preview – docs October 15, 2023 22:39 Inactive
@vercel vercel bot temporarily deployed to Preview – visualization October 15, 2023 22:40 Inactive
Copy link
Member

@manudeli manudeli left a comment

Choose a reason for hiding this comment

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

I left comment to edit some sentences as simple, commit suggestion please if you okay @minsoo-web

packages/react/src/ErrorBoundary.ko.mdx Outdated Show resolved Hide resolved
packages/react/src/ErrorBoundary.en.mdx Outdated Show resolved Hide resolved
@vercel vercel bot temporarily deployed to Preview – visualization October 15, 2023 23:17 Inactive
minsoo-web and others added 2 commits October 16, 2023 08:41
Co-authored-by: Jonghyeon Ko <[email protected]>
Co-authored-by: Jonghyeon Ko <[email protected]>
@minsoo-web
Copy link
Member Author

I left comment to edit some sentences as simple, commit suggestion please if you okay @minsoo-web

It is applied, thanks !!

@vercel vercel bot temporarily deployed to Preview – visualization October 15, 2023 23:43 Inactive
@manudeli
Copy link
Member

Better examples thanks!

@manudeli manudeli merged commit 90da45f into react/fix/useErrorBoundary-to-stable Oct 15, 2023
11 checks passed
@manudeli manudeli deleted the advise/useErrorBoundaryProps branch October 15, 2023 23:58
manudeli added a commit that referenced this pull request Oct 18, 2023
* feat(react): add useErrorBoundaryFallbackProps

* chore: update

* Create two-weeks-serve.md

* Update packages/react/src/ErrorBoundary.ko.mdx

Co-authored-by: 김석진 <[email protected]>

* Update packages/react/src/ErrorBoundary.ko.mdx

Co-authored-by: 김석진 <[email protected]>

* Update packages/react/src/ErrorBoundary.ko.mdx

Co-authored-by: 김석진 <[email protected]>

* docs: update

* Update packages/react/src/ErrorBoundary.ko.mdx

Co-authored-by: 김석진 <[email protected]>

* Update packages/react/src/ErrorBoundary.en.mdx

Co-authored-by: 김석진 <[email protected]>

* advise(react/errorboundary): add & change example case (#221)

* docs(errorboundary): change & add example case for ErrorBoundary

* docs(react/errorboundary): change & add example case to ErrorBoundary (en)

* docs(react): suggestion to advise/useErrorBoundaryProps (#227)

* Update ErrorBoundary.ko.mdx

Co-authored-by: Jonghyeon Ko <[email protected]>

* Update ErrorBoundary.en.mdx

Co-authored-by: Jonghyeon Ko <[email protected]>

---------

Co-authored-by: Jonghyeon Ko <[email protected]>

---------

Co-authored-by: 김석진 <[email protected]>
Co-authored-by: 김민수 <[email protected]>
manudeli added a commit that referenced this pull request Aug 3, 2024
* feat(react): add useErrorBoundaryFallbackProps

* chore: update

* Create two-weeks-serve.md

* Update packages/react/src/ErrorBoundary.ko.mdx

Co-authored-by: 김석진 <[email protected]>

* Update packages/react/src/ErrorBoundary.ko.mdx

Co-authored-by: 김석진 <[email protected]>

* Update packages/react/src/ErrorBoundary.ko.mdx

Co-authored-by: 김석진 <[email protected]>

* docs: update

* Update packages/react/src/ErrorBoundary.ko.mdx

Co-authored-by: 김석진 <[email protected]>

* Update packages/react/src/ErrorBoundary.en.mdx

Co-authored-by: 김석진 <[email protected]>

* advise(react/errorboundary): add & change example case (#221)

* docs(errorboundary): change & add example case for ErrorBoundary

* docs(react/errorboundary): change & add example case to ErrorBoundary (en)

* docs(react): suggestion to advise/useErrorBoundaryProps (#227)

* Update ErrorBoundary.ko.mdx

Co-authored-by: Jonghyeon Ko <[email protected]>

* Update ErrorBoundary.en.mdx

Co-authored-by: Jonghyeon Ko <[email protected]>

---------

Co-authored-by: Jonghyeon Ko <[email protected]>

---------

Co-authored-by: 김석진 <[email protected]>
Co-authored-by: 김민수 <[email protected]>
manudeli added a commit that referenced this pull request Aug 3, 2024
* feat(react): add useErrorBoundaryFallbackProps

* chore: update

* Create two-weeks-serve.md

* Update packages/react/src/ErrorBoundary.ko.mdx


* Update packages/react/src/ErrorBoundary.ko.mdx


* Update packages/react/src/ErrorBoundary.ko.mdx


* docs: update

* Update packages/react/src/ErrorBoundary.ko.mdx


* Update packages/react/src/ErrorBoundary.en.mdx


* advise(react/errorboundary): add & change example case (#221)

* docs(errorboundary): change & add example case for ErrorBoundary

* docs(react/errorboundary): change & add example case to ErrorBoundary (en)

* docs(react): suggestion to advise/useErrorBoundaryProps (#227)

* Update ErrorBoundary.ko.mdx


* Update ErrorBoundary.en.mdx


---------


---------
manudeli added a commit that referenced this pull request Aug 3, 2024
* feat(react): add useErrorBoundaryFallbackProps

* chore: update

* Create two-weeks-serve.md

* Update packages/react/src/ErrorBoundary.ko.mdx

Co-authored-by: 김석진 <[email protected]>

* Update packages/react/src/ErrorBoundary.ko.mdx

Co-authored-by: 김석진 <[email protected]>

* Update packages/react/src/ErrorBoundary.ko.mdx

Co-authored-by: 김석진 <[email protected]>

* docs: update

* Update packages/react/src/ErrorBoundary.ko.mdx

Co-authored-by: 김석진 <[email protected]>

* Update packages/react/src/ErrorBoundary.en.mdx

Co-authored-by: 김석진 <[email protected]>

* advise(react/errorboundary): add & change example case (#221)

* docs(errorboundary): change & add example case for ErrorBoundary

* docs(react/errorboundary): change & add example case to ErrorBoundary (en)

* docs(react): suggestion to advise/useErrorBoundaryProps (#227)

* Update ErrorBoundary.ko.mdx

Co-authored-by: Jonghyeon Ko <[email protected]>

* Update ErrorBoundary.en.mdx

Co-authored-by: Jonghyeon Ko <[email protected]>

---------

Co-authored-by: Jonghyeon Ko <[email protected]>

---------

Co-authored-by: 김석진 <[email protected]>
Co-authored-by: 김민수 <[email protected]>
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.

2 participants