Skip to content

Course 04 - Feature 254 dalia metadata #254 - #265

Merged
vismayajochem merged 3 commits into
productionfrom
feature-254-dalia-metadata
Oct 22, 2025
Merged

Course 04 - Feature 254 dalia metadata #254#265
vismayajochem merged 3 commits into
productionfrom
feature-254-dalia-metadata

Conversation

@amanda-wein

Copy link
Copy Markdown
Contributor

Summary of the discussion

Updated metadata for DALIA #254

Type of change (CHANGELOG.md)

Updated

  • Metadata updated for course 04

Workflow checklist

Automation

Closes #

PR-Assignee

Reviewer

  • 🐙 Follow the Reviewer Guidelines
  • 🐙 Provided feedback and show sufficient appreciation for the work done

@amanda-wein
amanda-wein requested review from Ludee and han-f October 15, 2025 12:59
Comment thread docs/courses/04_upload.md
- Contact: oedb@oeko.de
- License: [CC BY 4.0](https://creativecommons.org/licenses/by/4.0/deed.en)
- Attribution: Öko-Institut (2022): Upload - Contribute data to the OEP.
- Attribution: Öko-Institut (2023): Upload and Publish - Contribute data to the OEP.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this should stay as 2022?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I wasn't sure, looking back through the change history for the file the oldest date I saw was in 2023 (I used that for the publication date in the metadata). Do we have a 2022 publication date for this one recorded somewhere?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ah ok, then it might have been a typo. Thanks for sifting through. I can then approve.

han-f
han-f previously requested changes Oct 15, 2025

@han-f han-f left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I have a minor comment.

@han-f
han-f dismissed their stale review October 15, 2025 13:05

Comment was resolved

@Ludee Ludee changed the title Course 04 - Feature 254 dalia metadata Course 04 - Feature 254 dalia metadata #254 Oct 15, 2025
Comment thread docs/courses/04_upload.md Outdated

@Ludee Ludee left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved with suggestions

Co-authored-by: Ludwig Hülk <ludwig.huelk@rl-institut.de>
@vismayajochem vismayajochem mentioned this pull request Oct 21, 2025
5 tasks
@vismayajochem
vismayajochem merged commit a91c59e into production Oct 22, 2025
@vismayajochem

Copy link
Copy Markdown
Contributor

Everything looks good

@wingechr
wingechr deleted the feature-254-dalia-metadata branch November 29, 2025 17:27
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.

4 participants