Skip to content

Alison Z Frontend PR#22

Open
AlisonZ wants to merge 4 commits intopce-uw-jscript400:masterfrom
AlisonZ:master
Open

Alison Z Frontend PR#22
AlisonZ wants to merge 4 commits intopce-uw-jscript400:masterfrom
AlisonZ:master

Conversation

@AlisonZ
Copy link

@AlisonZ AlisonZ commented Aug 21, 2019

No description provided.


* **Your Answer:**
* **Your Answer:**
CORs error. It's a security check because we don't want to openly allow servers to ask separate browsers for information or send requests. Because we have a separate server for the backend than the frontend, need to mark our backend server as an allowed actor to request info.
Copy link

Choose a reason for hiding this comment

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

👍


* **Your Answer:**
* **Your Answer:**
The code is checking to see if we are in the development server or production.
Copy link

Choose a reason for hiding this comment

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

👍

* **Question:** What changes on the page after you successfully login? Why?

* **Your Answer:**
There are more menu items such as All Users and Create a New Post. This is because the Navigation Component is a ternary that shows one menu bar when not logged in and another when logged it. By passing down a logged in state, it renders the logged in Menu Bar that we see.
Copy link

Choose a reason for hiding this comment

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

👍

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