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

develop → main #63

Merged
merged 2 commits into from
Aug 30, 2023
Merged

develop → main #63

merged 2 commits into from
Aug 30, 2023

Conversation

kamo88
Copy link
Owner

@kamo88 kamo88 commented Aug 30, 2023

PR description

  • Add test case
  • Change README
capture
before after

Links

issue: #xxx

What you need to review

  • changes

others

others (ex: What we are not doing with this pull request

* refactor: Add test case
Add early return

* refactor: Add test case connect useDialog dialog element

* refactor: Add early return useDialogPromise

* refactor: Comment out test case

* refactor: Change test file extention

* refactor: Change data-testId → data-testid

* refactor: Add useDialogPromise connect dialog test case
Remove unused function

* refactor: Add test case in useDialogPromise

* refactor: Change prepare test in Dialog Copmponent

* refactor: Add DialogContainer handling test case
* docs: Add Component`s usage

* chore: version up
@kamo88 kamo88 self-assigned this Aug 30, 2023
@kamo88 kamo88 changed the title develop → master develop → main Aug 30, 2023
@kamo88 kamo88 merged commit ef2172e into main Aug 30, 2023
4 checks passed
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.

1 participant