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

Updated database docs and cheatsheet #293

Merged
merged 4 commits into from
Oct 25, 2018
Merged

Updated database docs and cheatsheet #293

merged 4 commits into from
Oct 25, 2018

Conversation

mattlsp
Copy link
Contributor

@mattlsp mattlsp commented Oct 23, 2018

No description provided.

@mattlsp mattlsp added the DOCS label Oct 23, 2018
@mattlsp mattlsp self-assigned this Oct 23, 2018
@mattlsp mattlsp requested review from Idered and mkucharz October 23, 2018 07:22
- The name of the class
- A description (optional)
- A schema
> To make use of the database you need to know what is the socket. If you don't know click [here](/getting-started/sockets)
Copy link
Member

Choose a reason for hiding this comment

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

let's use "Socket" with capital letter everywhere


**Data Classes** are templates for data objects you want to store on Syncano. In order to be able to [add Data Objects], you have to define a **Data Class** for that type of data object. To [create a Data Class] you have to provide:
# Defining
Copy link
Member

Choose a reason for hiding this comment

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

Maybe more descriptive e.g. "Defining data classes" ?

@codecov-io
Copy link

codecov-io commented Oct 24, 2018

Codecov Report

Merging #293 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #293   +/-   ##
=======================================
  Coverage   38.47%   38.47%           
=======================================
  Files          88       88           
  Lines        2773     2773           
  Branches        6        6           
=======================================
  Hits         1067     1067           
  Misses       1706     1706

@mattlsp
Copy link
Contributor Author

mattlsp commented Oct 24, 2018

Both changes done.

@mattlsp mattlsp merged commit 93743b6 into master Oct 25, 2018
@mattlsp mattlsp deleted the database_update branch October 25, 2018 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants