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

Fix browser example configuration #7

Merged
merged 8 commits into from
Dec 7, 2022

Conversation

magnetised
Copy link
Contributor

@magnetised magnetised commented Oct 31, 2022

see electric-sql/typescript-client#26

Copy link
Contributor

@balegas balegas left a comment

Choose a reason for hiding this comment

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

Is this required for current typescript-client, or just with electric-sql/typescript-client#26?

@balegas
Copy link
Contributor

balegas commented Oct 31, 2022

I suppose from the comment it is just with the PR.
Please:

  • merge the first one when ready
  • make a new release
  • update the examples in this PR to use the new release

@magnetised magnetised requested a review from balegas November 9, 2022 11:51
Copy link
Contributor

@balegas balegas left a comment

Choose a reason for hiding this comment

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

All good on my side.
Please bump the dependency version before merging.

@magnetised magnetised force-pushed the magnetised/configuration-refactor branch from cf39fd8 to 451af6d Compare November 30, 2022 16:31
@magnetised magnetised force-pushed the magnetised/configuration-refactor branch from 451af6d to 7f7e9e3 Compare December 1, 2022 10:44
@magnetised
Copy link
Contributor Author

@balegas think this is now good to go

@thruflo
Copy link
Contributor

thruflo commented Dec 6, 2022

Is this good to merge?

@balegas
Copy link
Contributor

balegas commented Dec 6, 2022

All good from my side Garry. Please merge.

@balegas balegas merged commit 4464c3c into main Dec 7, 2022
@thruflo thruflo deleted the magnetised/configuration-refactor branch April 3, 2023 14:13
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