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

Gh/27 show mjml errors #36

Merged
merged 4 commits into from
Jul 19, 2022
Merged

Gh/27 show mjml errors #36

merged 4 commits into from
Jul 19, 2022

Conversation

alexfarrill
Copy link
Collaborator

@monicakogler will you give some feedback on making the error state look better. Thanks!

image

@alexfarrill
Copy link
Collaborator Author

alexfarrill commented Jul 18, 2022

Unfortunately the line number indicated only corresponds directionally to the true line number

@psugihara
Copy link
Collaborator

Hm weird, maybe it's the line number from the compiled mjml rather than the react-mjml.

@alexfarrill alexfarrill merged commit 17986a5 into main Jul 19, 2022
@elliothursh elliothursh deleted the gh/27-show-mjml-errors branch October 9, 2022 16:51
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.

2 participants