Skip to content

fix(transform): Raise if restricted record file has no status - #657

Closed
sakshamarora1 wants to merge 1 commit into
CERNDocumentServer:masterfrom
sakshamarora1:fix/migration_errors
Closed

sakshamarora1 wants to merge 1 commit into
CERNDocumentServer:masterfrom
sakshamarora1:fix/migration_errors

Conversation

@sakshamarora1

Copy link
Copy Markdown
Contributor
  • Fixes for the case when a stream is marked as restricted, the record will be migrated as restricted but it could be that the file has no status set, which would make the parent access grant have no permissions
  • Fix check for restricted bool returning True if string was not empty
  • Fix RDM errors not being propagated to the log

* Fixes for the case when a stream is marked as restricted, the record will be migrated as restricted but it could be that the file has no status set, which would make the parent access grant have no permissions
* Fix check for restricted bool returning True if string was not empty
* Fix RDM errors not being propagated to the log
@sakshamarora1

Copy link
Copy Markdown
Contributor Author

Discussed with Zach IRL:
This will not be needed since records in a community can be granted access via community view grant.

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.

1 participant