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

Handle users' copy paste better. #11

Open
hrishikeshs opened this issue Jul 19, 2014 · 1 comment
Open

Handle users' copy paste better. #11

hrishikeshs opened this issue Jul 19, 2014 · 1 comment
Assignees
Labels

Comments

@hrishikeshs
Copy link
Owner

The parsing logic has a bug where it does not handle the case where a string value field of a json document is not updated properly when it contains a string that has : in it. An example of this would be:

{
 name: "mongo",
 type: "db: jsondb"
}

This will not update the document correctly.

@hrishikeshs hrishikeshs self-assigned this Jul 19, 2014
@hrishikeshs
Copy link
Owner Author

The above example works fine on the latest master. But when users copy paste json from external sources, the editor messes things a little. I'm updating the bug to reflect that.

@hrishikeshs hrishikeshs changed the title Editing does not work when a string used as a value has : in it. <del> Editing does not work when a string used as a value has : in it.</de> Handle users' copy paste better. Jul 31, 2014
@hrishikeshs hrishikeshs changed the title <del> Editing does not work when a string used as a value has : in it.</de> Handle users' copy paste better. <del> Editing does not work when a string used as a value has : in it.</del> Handle users' copy paste better. Jul 31, 2014
@hrishikeshs hrishikeshs changed the title <del> Editing does not work when a string used as a value has : in it.</del> Handle users' copy paste better. ~~Editing does not work when a string used as a value has : in it. ~~ Handle users' copy paste better. Jul 31, 2014
@hrishikeshs hrishikeshs changed the title ~~Editing does not work when a string used as a value has : in it. ~~ Handle users' copy paste better. Handle users' copy paste better. Jul 31, 2014
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

1 participant