-
Notifications
You must be signed in to change notification settings - Fork 2
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
Initial Design document finalize #1
base: main
Are you sure you want to change the base?
Conversation
@tekrajchhetri - take a look at the embargo and other design docs in dandi-archive and update this to reflect that. break the document into:
for the sequence diagram, insert the mermaid code directly in the markdown. please work directly with @puja-trivedi on the design and ping @djarecka for reviews. |
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.
@tekrajchhetri - I've left bunch of comments/questions. Basically, I asked all the questions I had when reading since many points were very general. However, I'm not sure if you are ready to answer all the questions. If something will have to be specified later (e.g. after seeing data, or talking to other people), perhaps it should be clarified?
fyi. i have not checked the architecture
Thanks @tylenolncuff, I will update the design document accordingly. |
Co-authored-by: Isaac To <[email protected]>
Co-authored-by: Isaac To <[email protected]>
Co-authored-by: Isaac To <[email protected]>
Co-authored-by: Isaac To <[email protected]>
Co-authored-by: Isaac To <[email protected]>
Co-authored-by: Isaac To <[email protected]>
Co-authored-by: Isaac To <[email protected]>
@candleindark You would need to edit using the Microsoft Visio. If you have access to Office 365 then you should have also access to the visio. I have shared the link with you via slack. |
I have updated the image now as per your comment |
Great. After some poking around, I found out my access for Office365 through Dartmouth. Thanks. |
This PR adds the following:
@satra