-
Notifications
You must be signed in to change notification settings - Fork 191
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
Docs: add info on export format to internals section #4467
Conversation
Hi Francisco, can you also remove the old content from the previous source (I think it was in |
Ok; my original idea was that I could anyways wait until all associated new content was merged and then delete all files in a given "remove" issue at once (seemed better than have each one decide what part to remove of the files), but from the message in the slack and this comment I assume you prefer that each deletes the part that each copies. PD: It seems that we are having a BIG bottleneck in the CI tests. Maybe we should discuss if there is something that can be done about this in the afternoon meeting... |
8e9bf5a
to
b09b747
Compare
I updated all references to "export" into "AiiDA archive". There are some inconsitencies due to the references in the actual code ( Now should be ready to review @giovannipizzi @sphuber |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor suggestions.
Also, please add a redirect from the old URL describing the data storage to this new URL.
4ecf508
to
bbbaae7
Compare
@giovannipizzi for now the redirection works for the full page (so |
For which the answer is that apparently this can't be done, so I guess this is the best we can do. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Fix #4037
Partially addresses #4450