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

Aggregate docs from new repo #61

Merged
merged 3 commits into from
Nov 9, 2023
Merged

Conversation

aaperis
Copy link
Contributor

@aaperis aaperis commented Nov 6, 2023

Describe the pull request:

  • Bug fix
  • Functional change
  • New feature
  • Code cleanup
  • Build system change
  • Documentation change
  • Language translation

Pull request long description:

This PR links the documentation aggregation logic from the sda-pipeline to the sensitive-data-archive repository. Only docs under sda/cmd are linked here since these correspond to components that are merged in and thus expected to get updates.

Some notes to keep in mind:

  • backup.md is no longer needed since in the current version of the stack in sensitive-data-archive the backup service has been partially merged with finalize in a way compliant with the FEGA usecase. This change needs to be addressed in the docs throughout.

  • notify.md and pipeline.md may be added later after they are migrated under sda/cmd and updated accordingly.

Related issues:

Related issue for migration of docs to the sensitive-data-archive repo: neicnordic/sensitive-data-archive#399 .

Release note:

Aggregate documentation updates from the sensitive-data-archive repository for ingest,verify,finalize,mapper and intercept.

@aaperis aaperis requested a review from a team November 6, 2023 09:02
@aaperis aaperis marked this pull request as ready for review November 8, 2023 13:14
@blankdots blankdots requested a review from a team November 9, 2023 12:46
@aaperis aaperis merged commit a72c150 into master Nov 9, 2023
3 checks passed
@aaperis aaperis deleted the aggregate-docs-from-new-repo branch November 9, 2023 12:48
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