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

remove data orchestrator and openrdm docs #611

Merged
merged 6 commits into from
Oct 12, 2023
Merged

remove data orchestrator and openrdm docs #611

merged 6 commits into from
Oct 12, 2023

Conversation

andrea-manzi
Copy link
Contributor

…for now)

Summary


Related issue :

@andrea-manzi andrea-manzi added the safe for preview Pull request vetted as safe for preview label Sep 22, 2023
@github-actions
Copy link

Documentation preview deployed!

Available at https://docs.egi.eu/documentation/611

Copy link
Member

@gwarf gwarf left a comment

Choose a reason for hiding this comment

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

I'm not fond of keeping old/legacy/unused cruft, If we are not sure we will use this again, we could just delete them, in the end they will always still be in the git history, should we need to revive them at some point.

@andrea-manzi andrea-manzi changed the title remove data orchestrator and openrdm docs( put existing doc as draft) remove data orchestrator and openrdm docs Sep 25, 2023
@github-actions
Copy link

Documentation preview deployed!

Available at https://docs.egi.eu/documentation/611

Copy link
Member

@gwarf gwarf left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions

This comment has been minimized.

@github-actions
Copy link

Documentation preview deployed!

Available at https://docs.egi.eu/documentation/611

@gwarf
Copy link
Member

gwarf commented Oct 12, 2023

@glarocca, @enolfc, @andrea-manzi should we just merge this now?

@github-actions

This comment has been minimized.

@github-actions
Copy link

Documentation preview deployed!

Available at https://docs.egi.eu/documentation/611

@andrea-manzi
Copy link
Contributor Author

yes :-)

@github-actions
Copy link

Documentation preview deployed!

Available at https://docs.egi.eu/documentation/611

@gwarf
Copy link
Member

gwarf commented Oct 12, 2023

@glarocca can you please approve this change?

Copy link
Contributor

@glarocca glarocca left a comment

Choose a reason for hiding this comment

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

LGTM

@gwarf gwarf merged commit ac3da05 into main Oct 12, 2023
12 checks passed
@gwarf gwarf deleted the data_services_update branch October 12, 2023 09:39
gwarf added a commit that referenced this pull request Oct 12, 2023
As EGI-ACE is over we keep the docs for the services in the EGI
portfolio. EC3 moves into IM and the portal for EC3 is removed as it is
to be deprecated.

---

**Related issue :** #611

Co-authored-by: Baptiste Grenier <[email protected]>
enolfc pushed a commit that referenced this pull request Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe for preview Pull request vetted as safe for preview
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants