Skip to content

Conversation

@slint
Copy link
Member

@slint slint commented Sep 25, 2025

CleanShot 2025-09-25 at 13 52 29@2x CleanShot 2025-09-25 at 13 52 54@2x CleanShot 2025-09-25 at 13 53 03@2x

@slint slint moved this to In review 🔍 in Sprint Q4/2025 Sep 25, 2025
@carlinmack
Copy link
Contributor

can the API still be changed? I feel like is_head should be is_latest

@slint
Copy link
Member Author

slint commented Sep 25, 2025

can the API still be changed? I feel like is_head should be is_latest

I would say yes, since there are basically only 2-3 users that know about it (and I'm not sure if they care about the payload). We could add is_latest and "deprecate" the old field and communicate this to them.

| `removal_date` | Date the record was removed |
| `citation_text` | Citation text (if available) |

## Streaming processing example
Copy link
Member

Choose a reason for hiding this comment

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

Nice idea to include an example!

@slint slint merged commit ecfa228 into zenodo:master Sep 25, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from In review 🔍 to To release 🤖 in Sprint Q4/2025 Sep 25, 2025
@slint slint deleted the metadata-dumps branch September 25, 2025 12:35
@slint
Copy link
Member Author

slint commented Sep 25, 2025

can the API still be changed? I feel like is_head should be is_latest

I would say yes, since there are basically only 2-3 users that know about it (and I'm not sure if they care about the payload). We could add is_latest and "deprecate" the old field and communicate this to them.

This means code changes, so I wouldn't want a deployment to block publishing the docs for the existing functionality. I opened zenodo/zenodo-rdm#1251 for this (and possibly others)

@slint slint moved this from To release 🤖 to Released ✔️ in Sprint Q4/2025 Sep 30, 2025
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.

3 participants