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

Implement better admin UI for modifying unit/resource opening hours #25

Closed
juyrjola opened this issue Oct 21, 2015 · 4 comments
Closed
Assignees

Comments

@juyrjola
Copy link
Contributor

No description provided.

@juyrjola
Copy link
Contributor Author

Some documentation on how the opening hours algorithm and data model is supposed to work:

https://gist.github.com/tituomin/88352932c9c83fa0fd7f

@akx
Copy link
Contributor

akx commented Oct 27, 2015

@tituomin, @juyrjola : Could someone write updated docs for the opening hours algorithm and data model? I understood the above gist is out of date.

@jussiarpalahti
Copy link
Member

@akx, this spec was written originally for https://github.com/libraries-fi/Library-Directory-Admin back in the day.

It explains the JSON document model stored in ElasticSearch. Respa has the same data (sort of) modeled as in Django ORM, plus some modifications/simplifications.

Since I made the newest implementation it might behoove me to write its docs.

Question is, should I document the ORM side, or merely the conceptual model, or how to make use of the data in db? Anything specific you'd like to docs to clarify?

@akx akx self-assigned this Oct 28, 2015
akx added a commit to akx/respa that referenced this issue Oct 28, 2015
It's not perfect yet, but it works.

Refs City-of-Helsinki#25
akx added a commit to akx/respa that referenced this issue Oct 28, 2015
It's not perfect yet, but it works.

Nb.: resources/templates/admin/resources/period_inline.html is a copy of
grappelli/templates/admin/edit_inline/tabular.html with exactly one
`include` statement added. I wish it wasn't so, but Grappelli isn't
apparently quite made with extensibility in mind.

Refs City-of-Helsinki#25
akx added a commit to akx/respa that referenced this issue Oct 28, 2015
It's not perfect yet, but it works.

Nb.: resources/templates/admin/resources/period_inline.html is a copy of
grappelli/templates/admin/edit_inline/tabular.html with exactly one
`include` statement added. I wish it wasn't so, but Grappelli isn't
apparently quite made with extensibility in mind.

Refs City-of-Helsinki#25
@akx akx closed this as completed Oct 29, 2015
@akx akx removed the in progress label Oct 29, 2015
@akx
Copy link
Contributor

akx commented Oct 29, 2015

@jussiarpalahti: I think I kinda got the general idea when working on this and #40, so I'm all good :)

Doesn't mean that docs would go amiss though...

ezkat referenced this issue in City-of-Turku/respa Oct 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants