Skip to content

fix: update type, remove unused imports, define AuthEvent type#47

Merged
J0 merged 2 commits into
supabase:mainfrom
cardenasj97:cardenasj97/improvements-1
Oct 28, 2024
Merged

fix: update type, remove unused imports, define AuthEvent type#47
J0 merged 2 commits into
supabase:mainfrom
cardenasj97:cardenasj97/improvements-1

Conversation

@cardenasj97
Copy link
Copy Markdown
Contributor

@cardenasj97 cardenasj97 commented Aug 11, 2024

What kind of change does this PR introduce?

fix type, remove unused imports, fix comments

What is the current behavior?

What is the new behavior?

Additional context

Add any other context or screenshots.

Comment thread src/createServerClient.ts Outdated
@cardenasj97 cardenasj97 requested a review from hf September 4, 2024 02:50
@cardenasj97
Copy link
Copy Markdown
Contributor Author

cardenasj97 commented Oct 12, 2024

This PR was already answered and updated, can you take a look again @J0 @hf ?

Copy link
Copy Markdown
Contributor

@J0 J0 left a comment

Choose a reason for hiding this comment

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

Thanks @cardenasj97

@J0 J0 changed the title fix type, remove unused imports, define AuthEvent type fix: update type, remove unused imports, define AuthEvent type Oct 28, 2024
@J0 J0 merged commit 4f4a375 into supabase:main Oct 28, 2024
J0 pushed a commit that referenced this pull request Nov 11, 2024
🤖 I have created a release *beep* *boop*
---


## [0.5.2](v0.5.1...v0.5.2)
(2024-11-05)


### Bug Fixes

* add @types/cookies to dependencies
([#63](#63))
([47e5f16](47e5f16))
* check chunkedCookie is string in server client
([#57](#57))
([549fe62](549fe62))
* update type, remove unused imports, define AuthEvent type
([#47](#47))
([4f4a375](4f4a375))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants