Skip to content

w6-frontend#26

Open
wedge1126 wants to merge 7 commits intopce-uw-jscript400:masterfrom
wedge1126:master
Open

w6-frontend#26
wedge1126 wants to merge 7 commits intopce-uw-jscript400:masterfrom
wedge1126:master

Conversation

@wedge1126
Copy link
Copy Markdown

No description provided.

Comment thread README.md
* **Question:** What error do you get? Why?

* **Your Answer:**
* **Your Answer:** Access has been blocked by CORS policy. The front-end and back-end have different origins. Basically, they're hosted by different servers.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

Comment thread README.md
* **Question:** How could you convert this method to an `async` method?

* **Your Answer:**
```js
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

Comment thread README.md
* **Question:** When you click "Logout", nothing happens unless you refresh the page. Why not?

* **Your Answer:**
* **Your Answer:** logout doesn't call setState to trigger the React component lifecycle.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

Comment thread README.md
* **Question:** In what component did you add the `loading` property and why?

* **Your Answer:**
* **Your Answer:** I added to `components/users/Container.js` because Posts are subdocuments of Users and Users are loaded in this container.
Copy link
Copy Markdown

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