Skip to content
This repository has been archived by the owner on Oct 5, 2020. It is now read-only.

Compile and xslt for doc details #2

Merged
merged 5 commits into from
Jun 12, 2014
Merged

Compile and xslt for doc details #2

merged 5 commits into from
Jun 12, 2014

Conversation

ryan321
Copy link
Contributor

@ryan321 ryan321 commented Jun 12, 2014

I added the compile directive to support the xsmlt transform of doc details. I changed mlrest.js to take options for specifying which transform to use for the document get over the rest api.

@dmcassel dmcassel merged commit d15854e into marklogic-community:master Jun 12, 2014
@dmcassel
Copy link
Contributor

ryan, I took the compile.js and the transform, but left out the changes to detail.html. I see why you're going for that, and detail.html definitely needs to be more generic, but I'm not sold on using a server side transform for that. Let me kick that around, I may come back to this.

@ryan321
Copy link
Contributor Author

ryan321 commented Jun 12, 2014

The reason I did that is because it seemed like a good baseline is that the entire doc comes back in the rest call, but it is formatted enough that it can be viewed as html. For a project that just wants to get up and running, that seemed like a good start. If there's a desire to do something different there's still that possibility.
I didn't find a good approach to displaying the doc structure all on the client side, so I followed suit which what I've seen work.
This also gives the ability to not return the entire doc. You may want to hide some metadata or other confidential info.
But I'm for any better way.

Date: Thu, 12 Jun 2014 13:45:43 -0700
From: notifications@github.com
To: slush-marklogic-node@noreply.github.com
CC: ryan321@outlook.com
Subject: Re: [slush-marklogic-node] Compile and xslt for doc details (#2)

ryan, I took the compile.js and the transform, but left out the changes to detail.html. I see why you're going for that, and detail.html definitely needs to be more generic, but I'm not sold on using a server side transform for that. Let me kick that around, I may come back to this.


Reply to this email directly or view it on GitHub.

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

Successfully merging this pull request may close these issues.

3 participants