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

add events and news to decap #91

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

add events and news to decap #91

wants to merge 5 commits into from

Conversation

mingness
Copy link
Member

@mingness mingness commented Jan 31, 2025

events and news are now editable on decap with this PR.
note that news items are displayed by publish_date, but to be able to refer to event media, the start_date and event_title are also included in the metadata, so that the same media folder as an event can be referred to in a news item. note that if you want to access the media for an existing event, you'll need to create the news item, and then edit it again to see the media.

Copy link

github-actions bot commented Jan 31, 2025

PR Preview Action v1.6.0

🚀 View preview at
https://prachtsaal.github.io/prachtsaal.berlin/pr-preview/pr-91/

Built to branch gh-pages at 2025-02-05 10:52 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Copy link
Member

@hamoid hamoid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yay! exciting :-)

@hamoid
Copy link
Member

hamoid commented Jan 31, 2025

to show events as news, is it just by date?

@mingness
Copy link
Member Author

News items are the same as currently, they are a separate collection, and only will show up on the home page, or portfolio. I think of news as announcements, versus event descriptions. We might want to clean up what we have at the moment also.

@mingness
Copy link
Member Author

mingness commented Feb 1, 2025

@hamoid does the current logic and organization make sense to you, or do you think we should do something different?

@hamoid
Copy link
Member

hamoid commented Feb 5, 2025

Is there something odd in the Processing Community Catalog Reading? There are 4 images entries without images, only captions. In the preview the 4 captions are shown. Is that a bug?

Update: But the images actually do show up in the news entry. Maybe some mismatch in paths or names?

Update: I think because in the admin it's image and in the document it's file. Fixing it.

@hamoid
Copy link
Member

hamoid commented Feb 5, 2025

I find the overlap between news and events a bit confusing, but I don't know yet of a better organization. There's events accessible at the top. There's news, not accessible from the top. Then there's a list of news in the home page. There's no archive of news.

A layout that comes to mind is this: there's events and news, both in the top menu. Any event and news item can be marked to be shown in the home page. Events would be things with a date. News are things worth mentioning where the date is not so important: spring clean up, new member, someone got an award, etc.

Does that makes sense?

@mingness
Copy link
Member Author

mingness commented Feb 5, 2025

Your interpretation of the current state is not quite right. news can be archived in the portfolio as well. But what you mean is, there is no list of past news. But also, we don't have true list of all past events either.

I think of "news" more as announcements. events are descriptions of events.

announcements are more time sensitive, and hence are posted on the front page. Yet, we have a page just for documenting our events, so there are upcoming events, and then our portfolio.

@hamoid
Copy link
Member

hamoid commented Feb 5, 2025

What does it mean "news can be archived in the portfolio as well"?

News currently have required event title, event start date, event end date, so I can't save the open call for new tenant artists without those.

What's the difference between Title and Event Title?

Should event start date and event end date be news start date and news end date?

@hamoid
Copy link
Member

hamoid commented Feb 5, 2025

The video embed in "open call for new tenant artists" is too low level for anyone except us to edit. Maybe I should add an non-required field to point to a video file? And if present, the template puts the embed code?

@mingness
Copy link
Member Author

mingness commented Feb 5, 2025

The open calls are a different beast. I have not added them to decap.

news items can have the tag "portfolio" and be listed in the event portfolio. An example of an event that has both a news item and event item is the processing book launch. I'm not sure which one should be which one, but the documentation of the event is currently a news item, and the event signup is in the event description. I think this is not a good model of how we might organize information.

@mingness
Copy link
Member Author

mingness commented Feb 5, 2025

I added information to the PR description about the field event_title

the start_date and event_title are also included in the metadata, so that the same media folder as an event can be referred to in a news item.

news items are not using the news layout, but most are using the event layout, so that is why I've left the event_end_date is there as well.

for news items, title is the title of the news. event_title is the name of the event, so you can access the media for the event, and consolidate media.

@mingness
Copy link
Member Author

mingness commented Feb 5, 2025

Is there something odd in the Processing Community Catalog Reading? There are 4 images entries without images, only captions. In the preview the 4 captions are shown. Is that a bug?

Update: But the images actually do show up in the news entry. Maybe some mismatch in paths or names?

Update: I think because in the admin it's image and in the document it's file. Fixing it.

just now saw this. will check later. I will have to go dark for the rest of the work day.

@hamoid
Copy link
Member

hamoid commented Feb 5, 2025

The open calls are a different beast. I have not added them to decap.

But there is an open call in news :)

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

Successfully merging this pull request may close these issues.

2 participants