Conversation
Preview available
|
c5cb722 to
4e142d8
Compare
6f3d521 to
3eda497
Compare
1144d69 to
8e763d9
Compare
4225543 to
38b0d50
Compare
|
Adding do-not-merge for now. We can remove it once we're using the actual api. |
Sure we can keep it. Keep in mind that it's still draft, many things are in progress. |
feef3e6 to
7954c4a
Compare
7954c4a to
cc0860c
Compare
Co-authored-by: Mia Bajić <38294198+clytaemnestra@users.noreply.github.com>
|
I think that we can close also this ticket with the PR. |
There was a problem hiding this comment.
@nikoshell Looks good overall! Is there a reason why we changed entry.slug to entry.id?
Also, can you update the session page to not show Room and Start for now? They are not finalized yet.
Co-authored-by: Mia Bajić <38294198+clytaemnestra@users.noreply.github.com>
Co-authored-by: Mia Bajić <38294198+clytaemnestra@users.noreply.github.com>
The file() loader creates entries from a single file that contains an array of objects with a unique id field. |
|
|
Removing |
clytaemnestra
left a comment
There was a problem hiding this comment.
LGTM, thanks a lot!
hypha
left a comment
There was a problem hiding this comment.
Left a couple of minor things, none of which should block the merge. Thanks so much for working on this!


🖼️ Preview available 🖼️ : https://ep-website--1101.org.readthedocs.build/