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.

Sign up for GitHub

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

Jump to bottom

I have found an issue that the button does not respond after adding a scheme. #55

Closed
wuliqq opened this issue Apr 9, 2023 · 4 comments · Fixed by #56
Closed

I have found an issue that the button does not respond after adding a scheme. #55

wuliqq opened this issue Apr 9, 2023 · 4 comments · Fixed by #56

Comments

@wuliqq
Copy link

wuliqq commented Apr 9, 2023 •
edited

I have found an issue that the button does not respond after adding a scheme.

When I click the button[Generate SQL], the page does not respond.

image
@whoiskatrin
Copy link
Owner

whoiskatrin commented Apr 9, 2023 •
edited

@wuliqq provide please all the input data. any errors in the console?

@saaagarsingh
Copy link

Hey @whoiskatrin if you can explain me more about the issue here then I would love to resolve it. Right now it not much of understandable what exactly the issue is.

@wuliqq
Copy link
Author

wuliqq commented Apr 10, 2023

@wuliqq provide please all the input data. any errors in the console?

I have identified the issue. When I add an extra newline character at the end of the scheme content, it causes the page to crash. However, when I remove the newline character, the issue disappears.

@wuliqq
Copy link
Author

wuliqq commented Apr 10, 2023

@wuliqq provide please all the input data. any errors in the console?

But I don't know why. I also tried https://www.sqltranslate.app/ and the same issue occurred.

difagume added a commit to difagume/sql-translator that referenced this issue Apr 12, 2023
@difagume
solution incidence whoiskatrin#55, type is added to the Schema button…
dd13167 
… to avoid submit
@difagume difagume mentioned this issue Apr 12, 2023
Merged
whoiskatrin added a commit that referenced this issue Apr 13, 2023
@whoiskatrin
Merge pull request #56 from difagume/main
044df53 
Fix issue #55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix issue #55 difagume/sql-translator
3 participants
@whoiskatrin @wuliqq @saaagarsingh