Skip to content

3 eln/tutorial eln/review from CD (so far)#2

Open
caro-demi wants to merge 5 commits into
mainfrom
3-eln/tutorial-ELN/review
Open

3 eln/tutorial eln/review from CD (so far)#2
caro-demi wants to merge 5 commits into
mainfrom
3-eln/tutorial-ELN/review

Conversation

@caro-demi
Copy link
Copy Markdown
Collaborator

overall comments to tutorial. exprimental step - not capitalised, but Lab Notebook capitalised.
mentioned important definitions can be linked to Wiki (Lab Notebook, Inventory, Entity, Space, Project, etc.)
Figure 6: (different description in screenshot and in playground.)
Detailed information about rights and roles can be found in the Data Store wiki. - link is missing
Figure 11, 12, 13: ( figures need to be updated, no collection Structure_TEM)
Fig. 12 and 13 are not described in text and purpose of them is not clear.
Masterdata definitions process - link to Wiki
Figure 14, 15, 16 - needs to be updated, no object Demo_001.

This is a review so far, will continue on monday.

caro-demi added 2 commits May 22, 2026 15:34
…k capitalised.

mentioned important definitions can be linked to Wiki (Lab Notebook, Inventory, Entity, Space, Project, etc.)
Figure 6: (different description in screenshot and in playground.)
Detailed information about rights and roles can be found in the Data Store wiki. - link is missing
Figure 11, 12, 13: ( figures need to be updated, no collection Structure_TEM)
Fig. 12 and 13 are not described in text and purpose of them is not clear.
Masterdata definitions process - link to Wiki
Figure 14, 15, 16 - needs to be updated, no object Demo_001
@caro-demi caro-demi requested a review from angelaari May 22, 2026 17:29
Copy link
Copy Markdown
Collaborator

@angelaari angelaari left a comment

Choose a reason for hiding this comment

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

The Branch shows Merge conflicts, please resolve before sending request for next review

Comment thread 3-eln/tutorial-ELN.html Outdated
Comment thread 3-eln/tutorial-ELN.html Outdated
@caro-demi caro-demi requested a review from angelaari May 26, 2026 14:35
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.

2 participants