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

Add feature of JSON to XML to online tool #79

Open
amitguptagwl opened this Issue Apr 26, 2018 · 20 comments

Comments

Projects
None yet
4 participants
@amitguptagwl
Member

amitguptagwl commented Apr 26, 2018

This issue is raised for developers who are just starting with open source development. So please refrain yourself if you're experienced.

Please check README or spec folder for examples.

Online demo tool demonstrate what XML parser can do. However it is currently just parsing XML to JSON and validating XML only. As a part of this change, we have to parse JSON to XML;

You may discuss here about design changes.

Check how to.

@geekiam23

This comment has been minimized.

geekiam23 commented Apr 26, 2018

I'd like to give it a shot.

@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented Apr 26, 2018

You'r welcome @geekiam23

@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented May 22, 2018

@geekiam23 any update on this issue? Is it fine if any other contributor can pick this issue?

@geekiam23

This comment has been minimized.

geekiam23 commented May 22, 2018

@amitguptagwl Yeah, I couldn't figure it out. Please let someone else give it a shot.

@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented Jun 5, 2018

This issue can be picked by any contributor now. Demo UI has been simplified. So now it is just the matter of 10 mins change.

@sakshivij

This comment has been minimized.

sakshivij commented Jun 14, 2018

Hey! I would like to work on this issue.

@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented Jun 14, 2018

Yes, please go on.

@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented Jun 22, 2018

@sakshivij have you got the chance to work on?

@sakshivij

This comment has been minimized.

sakshivij commented Jun 22, 2018

Yes! I am working on it. I just need this weekend. If I'm not able to resolve the issue, I'll let you know by Sunday.

@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented Jun 22, 2018

No problem, take your time. It's not urgent. I just wanted to check if you have stuck in some issues and need help

@sakshivij

This comment has been minimized.

sakshivij commented Jul 1, 2018

Hey!
Thank you so much for understanding.
I have tried my best but I am not just not able to figure out the problem with the code. The code works just fine on console. But on browser, it just doesn't work.The browser doesn't show any error. I am stuck and don't know how to proceed further.Could you please help and point me out where I must be doing wrong?

@sakshivij

This comment has been minimized.

sakshivij commented Jul 1, 2018

A little bit of hint would be great!

@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented Jul 1, 2018

Umm.. what are you actually trying?

  1. Open the index.html
  2. Open the section which is showing options and buttons to parse XML to JSON
  3. Paste and create another section. Change the IDs. Modify the options as per the example snippet given on README.md
  4. Add javascript code to read options and call JSON to XML parser. It is present on index.html only.
@sakshivij

This comment has been minimized.

sakshivij commented Jul 1, 2018

Yes! I have done the same thing. But the code gives error.
image

@sakshivij

This comment has been minimized.

sakshivij commented Jul 1, 2018

I don't know but somehow the browser is not able to reference Parser. The code works fine on console.

@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented Jul 2, 2018

Hmm.. do one thing, install "sudo npm -g i http-server". Go to the git repo from command line and run this command "http-server". Open browser and hit this UTL "http://127.0.0.1:8080"

I hope you should not see the error.

@sakshivij

This comment has been minimized.

sakshivij commented Jul 2, 2018

Unfortunately error persists!!

@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented Jul 3, 2018

Are you seeing the errors here as well?

https://naturalintelligence.github.io/fast-xml-parser/

I've checked it on firefox and chrome on Ubuntu and it seems working fine.

Ah! I think I got it. You must be seeing this error after your changes. You should have used variable name in small letters "parser" not "Parser

@lfplazas10

This comment has been minimized.

Contributor

lfplazas10 commented Nov 8, 2018

Is this task still undone? If so, I would like to tackle it.

@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented Nov 8, 2018

@lfplazas10 Yes please go on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment