Skip to content

Various updates#9

Open
JAL9302 wants to merge 3 commits into
exavolt:masterfrom
JAL9302:master
Open

Various updates#9
JAL9302 wants to merge 3 commits into
exavolt:masterfrom
JAL9302:master

Conversation

@JAL9302

@JAL9302 JAL9302 commented Mar 3, 2012

Copy link
Copy Markdown

Adds a confirm box before deletions, and allows additionalProperties to correctly show up when the starting data is passed into renderObject has additionalProperties, and the schema contains additionalProperties.

Also adds a new helloworld html file demonstrating the new changes, specifically onde working with starting data containing additionalProperties.

@JAL9302

JAL9302 commented Mar 19, 2012

Copy link
Copy Markdown
Author

@exavolt you still supporting this project?

@exavolt

exavolt commented Mar 31, 2012

Copy link
Copy Markdown
Owner

@jodoglevy Ah. Yes. Sorry for the late response. Got occupied by other projects.

I'll review the request tomorrow.

@JAL9302

JAL9302 commented Apr 9, 2012

Copy link
Copy Markdown
Author

@exavolt I think the pull request you merged from @adamgreig fixes the additionalProperties issue I was trying to fix in this pull request. So you may not need that anymore (although I did not test to see if his code works for my helloworld test file included in the commits). But confirm on delete would still be useful considering hitting 'enter' when focused in an array currently deletes the current index that has focus (which I'm not sure is what it is meant to do), without prompt.

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