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

Test Pull display #9

Open
tlovett1 opened this issue Feb 4, 2015 · 3 comments
Open

Test Pull display #9

tlovett1 opened this issue Feb 4, 2015 · 3 comments

Comments

@tlovett1
Copy link
Owner

tlovett1 commented Feb 4, 2015

Hey @sc0ttkclark, I think the test pull thing is a good idea. However, I think we can do it a bit more elegantly. Any ideas?

@sc0ttkclark
Copy link
Contributor

Was just chatting with Dave Ross, we think we could pull up a new modal with the content listed there. Also, need to array_map( 'esc_html', ... ) all the feed post_field / meta_fields / taxonomy_fields data and convert that SimpleXMLElement on display so it's an escaped array of data too.

Currently using var_export but I think we could display it better just looping through it and outputting it in a nicer <ul>.

@tlovett1
Copy link
Owner Author

tlovett1 commented Feb 4, 2015

Definitely agree. Can you take care of this? No rush.

@sc0ttkclark
Copy link
Contributor

Sure, I've been meaning to dive into the modal init for these types of cases.

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

No branches or pull requests

2 participants