Skip to content
This repository has been archived by the owner on Mar 15, 2021. It is now read-only.

RFC0021: Archive resource #35

Merged
merged 5 commits into from
Oct 31, 2018
Merged

RFC0021: Archive resource #35

merged 5 commits into from
Oct 31, 2018

Conversation

arnau
Copy link
Contributor

@arnau arnau commented Sep 27, 2018

UPDATE: After @MatMoore question on the actual purpose of the archive resource, we agreed in removing it. The new RFC reflects this decision.

@arnau arnau added the rfc label Sep 27, 2018
@arnau
Copy link
Contributor Author

arnau commented Sep 27, 2018

You can find some more context in this thread https://github.com/openregister/registers-rfcs/pull/12/files#r185759983

@MatMoore
Copy link

I don't think I actually understand the use case for downloading entries and items as files in a zip. And also, from what I can see in google analytics this endpoint gets no traffic at the moment.

What if we just deprecated the endpoint instead?

@arnau
Copy link
Contributor Author

arnau commented Oct 25, 2018

@MatMoore If I understand correctly where this endpoint comes from, it is the original way to get a full copy for a register in a single request. Internally, instead, we use RSF but RSF is not part of the specification and so far the position is that it shouldn't be standardised.

So, if we remove this endpoint we accept that there is no relevant user need to get a copy for the register in a single request, which according to the metrics you mention is the case. I'm happy to discuss in person, perhaps with @gidsg so we have all points of view on the matter.

@MatMoore
Copy link

Sounds good 👍


## Motivation

Currently, the specification defines the Archive resource as a zip file withi

Choose a reason for hiding this comment

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

there's a typo here

Arnau Siches added 5 commits October 31, 2018 14:08
Signed-off-by: Arnau Siches <arnau.siches@digital.cabinet-office.gov.uk>
Signed-off-by: Arnau Siches <arnau.siches@digital.cabinet-office.gov.uk>
Signed-off-by: Arnau Siches <arnau.siches@digital.cabinet-office.gov.uk>
Signed-off-by: Arnau Siches <arnau.siches@digital.cabinet-office.gov.uk>
Signed-off-by: Arnau Siches <arnau.siches@digital.cabinet-office.gov.uk>
@arnau arnau merged commit 478d9f9 into master Oct 31, 2018
arnau pushed a commit that referenced this pull request Oct 31, 2018
@arnau arnau deleted the archive-resource branch October 31, 2018 14:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants