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

Save (Ctrl+S) feature doesn't work in some cases #480

Closed
ghost opened this issue Nov 16, 2022 · 4 comments
Closed

Save (Ctrl+S) feature doesn't work in some cases #480

ghost opened this issue Nov 16, 2022 · 4 comments
Assignees
Labels

Comments

@ghost
Copy link

ghost commented Nov 16, 2022

The save feature doesn't work in some cases.

  • I have 25 saved patterns in "My Patterns"
  • It always worked until this moment (so I'm not sure if it could be due to a regression in a recent update or it's just a bug)
  • I had an error (still visible in the screenshot) at the moment of the save in the regex (a forward slash not escaped).
    ⚠️ In this case, I'd say it's better to point out the error to the user, instead of just not saving the pattern.

When pressing the Save button nothing happens, as well as when pressing the keyboard shortcut Ctrl + S.
This is what I see when trying to save an invalid pattern in my account:
image

When I fixed the error in the regex, it finally saved the pattern in my profile.

⚠️ Still, I wasn't able to add a Text to test the new regex: when I add it, then it wouldn't save (again, and the problem persist after logout and login, and on different browsers - Chrome, Firefox, Edge).
The error is the same as reported when trying to save an invalid regex pattern.
This is the Text I'm trying to add to test the regex:

./../../XSD/VWXML_Structure.xsd

./../../XSD/VWXML_Behavior.xsd

./../../XML/ProjectProperties.xml

The regex I saved in my patterns is this one: ([A-Z\.\/]+)\/([A-Za-z_\.]+).

Thank you in advance for your support.

@gskinner
Copy link
Owner

Thanks for the report. At a quick glance, this is generating a server error:
Failed to load resource: the server responded with a status of 418 ()

We will look into why that's happening. It could be related to our recent server move.

@gskinner
Copy link
Owner

Quick update. Looks like it is related to the new hosting. We're continuing to investigate.

@gskinner
Copy link
Owner

This should be fixed now. It was an enhanced security setting on the new host.

@ghost
Copy link
Author

ghost commented Nov 17, 2022

This should be fixed now. It was an enhanced security setting on the new host.

Thank you very much for the super quick fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants