Skip to content

for-class#3

Open
seagreenmedia wants to merge 3 commits intocode-differently:mainfrom
seagreenmedia:main
Open

for-class#3
seagreenmedia wants to merge 3 commits intocode-differently:mainfrom
seagreenmedia:main

Conversation

@seagreenmedia
Copy link
Copy Markdown

No description provided.

seagreenmedia added 2 commits February 24, 2021 17:56
@kburd kburd self-requested a review March 1, 2021 19:55
CREATE TABLE IF NOT EXISTS pokemon (
pokemon_id INT PRIMARY KEY NOT NULL,
name VARCHAR(255) NOT NULL,
color VARCHAR(255),
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We generally dont want to store data related to viewing the data in the database. We should store this on the front end

pokemon_id INT PRIMARY KEY NOT NULL,
name VARCHAR(255) NOT NULL,
color VARCHAR(255),
type VARCHAR(255),
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We dont want to store the types this way. We want to use an many to many relationship

@@ -0,0 +1,47 @@
let password = "0507198Four";
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is a great start, we want to run this code when an express route is triggered now

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