-
Notifications
You must be signed in to change notification settings - Fork 0
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
Sort Related Documents Chronologically #161
Comments
Example: oscys.caseid.0249 |
These are not currently sorted by date because there is not a "date" field associated with them. It looks as though it is part of the title:
Compare that with
|
@jduss4 Is there a reason why there is no date field associated with them? The caseRelatedDocument documents are encoded exactly like the caseDocument. |
@kacinash I haven't looked into it too much, but I'm guessing it's just not coded into the tei -> solr script. I'll dig in a bit more! |
Notes for Jessica: |
This seems as though it is working. Can you confirm, @kacinash ? http://rosie.unl.edu/earlywashingtondc/cases/oscys.caseid.0001#rel_documents |
@jduss4, seems to be working! You can see it best here: http://rosie.unl.edu/earlywashingtondc/cases/oscys.caseid.0342 One thing I did notice is that the documents with no date have an (n.d.) after them. In Case Documents, they are just blank. |
I think that's happening because I was trying to co-opt some of the existing functionality of dates. What things should appear with n.d. after them? |
Nothing. It's just blank. See the bottom of http://rosie.unl.edu/earlywashingtondc/cases/oscys.caseid.0001#documents |
I mean are there any documents in that "related documents" list on cases that should appear with "n.d." ? If not, I'll just strip it out from anything. I think it's just a default right now if it can't find a date. |
No. We decided we didn't want "n.d." to appear anywhere. If there is no date, there is just nothing after the document name. |
Oh! Is this a sitewide decision? Because if so, then I can wipe out the "n.d." with impunity. :D |
Yup! |
I think that this and all the n.d.s and ()s, etc, are now taken care of! :) |
I think this got fixed on the case page, but on the people page, Related Documents are still not sorted in any way. |
Okay! |
@kacinash does this look like the people related documents are sorting the way you would like? http://rosie.unl.edu/earlywashingtondc/people/per.000003#documents Undated are appearing at the top, currently, which might be somewhat difficult to switch unless if we have dates descending rather than ascending |
I'll have to check, but I think that they are coming from different sources (one is the main solr doc, the others are the embedded json in the docs), but I'll have to look to see how I was sorting those. |
I'm going to move past this for now in the interests of getting to more issues today, but I'll leave it open so that I can circle back during the next bug squash! |
Eventually, I would like to sort Related Documents chronologically, so that you can get a sense of the chronological progression of the case like with the Case Documents.
The text was updated successfully, but these errors were encountered: