rerender after fetch POST and PUT request - reactjs

I have an app that GET's data from a REST API. I am able to fetch the data. I also need to be able to POST data to the API and PUT edits to the API. My code is able to successfully perform all of these operations, however I need to refresh the page to see the submissions. The trouble I am having is with automatically rerender the page upon submissions. I know I either have to setState or refetch the API after the call. I do not know how to structure this though. Any suggestions?
import React, { Component } from 'react';
import './App.css';
class App extends Component {
constructor(props) {
super(props);
this.state = {
dataSource: [],
isLoaded: false,
}
this.editBeersLikes = this.editBeersLikes.bind(this);
this.addNewBeer = this.addNewBeer.bind(this);
}
seeBeers() {
return fetch("https://beer.fluentcloud.com/v1/beer/")//specify id number to show single beer ex. "https://beer.fluentcloud.com/v1/beer/99"
.then(response => response.json())
.then(responseJson => {
this.setState({
isLoaded: true,
dataSource: responseJson,
});
return responseJson;
})
.catch(error => console.log(error)); //to catch the errors if any
}
addNewBeer() {
fetch("https://beer.fluentcloud.com/v1/beer/", {
body: "{\"name\":\"\",\"likes\":\"\"}",//input beer name and like amount ex "{\"name\":\"Michelob Ultra\",\"likes\":\"-5\"}"
headers: {
"Content-Type": "application/json"
},
method: "POST"
})
console.log("Beer Added!");
}
editBeersLikes() {
fetch("https://beer.fluentcloud.com/v1/beer/99", {//must specify id number to edit a single beer's likes ex "https://beer.fluentcloud.com/v1/beer/99"
body: "{\"likes\":\"\"}", //input amount of likes to edit ex "{\"likes\":\"22\"}"
headers: {
"Content-Type": "application/json",
"Accept": "application/json"
},
method: "PUT"
})
console.log("Likes Successfully Updated!");
}
componentDidMount() {
this.seeBeers();
//this.editBeersLikes(); //uncomment when you want to edit likes
//this.addNewBeer(); //uncomment when you want to add beers
}
render() {
return (
<div className="App">
<h1>What is in My Fridge?</h1>
<ul>
{this.state.dataSource.map(dataSource => {
return <li key={`dataSource-${dataSource.id}`}>{dataSource.name} | {dataSource.likes}</li>
})}
</ul>
</div>
);
}
}
export default App;

The easiest way is probably to re-fetch the data after you have successfully POSTed or PUT them to the server. That would also be the cleanest and least error prone way because you always get a fresh response from the server with the most recent set of data.
However, you could use a pattern that is often referred to as "optimistic ui", meaning that you keep two states: the current state and the state how it will look like if the server request is successful. You would then push the new or updated item to your dataSource state and show it in your UI but revert it to the old state from before the server request if the request fails.
The downside of the latter approach is that you sometimes don't know all the data (especially auto generated IDs or lastUpdated fields) without querying the server again.
So if you want a relatively easy, clear, and solid solution you can call this.seeBeers() after each POST/PUT request to receive an updated set of data.
Keep in mind that this might not always work as expected when you're using e.g. sharding or an elastic search layer for read operations as they might not be in sync directly after you finished the write operation. That's probably not relevant for you though, as that's usually only a challenge when working with large scale setups.

Related

React: Can't figure out how to use fetched API response

I'm working with a nice chatbot program for React someone wrote, and the thing is, you can actually bind the bot's responses to function calls like this:
render() {
const { classes } = this.props;
return (
<ChatBot
steps={[
{
...
{
id: '3',
message: ({ previousValue, steps }) => {
this.askAnswer(previousValue)
},
end: true,
},
]}
/>
);
Where message is the answer of the bot that it calculates based on the previousValue and askAnswer is a custom function you'd write. I'm using an API that inputs the previousValue to a GPT model, and I want to print the response of this API.
However, I just can't wrap my head around how I could pass the response of the API to the message. I'm trying this:
constructor(props) {
super(props);
this.state = { response: " " };
}
...
askAnswer(question) {
var jsonData = { "lastConversations": [question] }
fetch('http://my_ip:80/query', {
method: 'POST',
mode: 'cors',
headers: {
'Content-Type': 'application/json'
},
body: JSON.stringify(jsonData)
}).then(response => response.json())
.then(data => { this.setState({ response: data["botResponse"] }) });
return (this.state.response)
}
I've been struggling with this for the past 2-3 hours now.
I've tried a couple of combinations, and nothing seems to work:
If I do as seen above, it seems like this.state.response just won't get
updated (logging data["botResponse"] shows there is a correct
reply, so the API part works and I get the correct response).
If I try async-await for askAnswer and the fetch call, then I can
only return a Promise, which is then incompatible as input for the
ChatBot message.
If I do not await for the fetch to complete, then
this.state.response just stays the default " ".
If I return the correct data["botResponse"] from within the second .then after the fetch, nothing happens.
How am I supposed to get the API result JSON's data["botResponse"] text field out of the fetch scope so that I can pass it to message as a text? Or, how can I get a non-Promise string return after an await (AFAIK this is not possible)?
Thank you!
In your last 2 line
.then(data => { this.setState({ response: data["botResponse"] }) });
return (this.state.response)
You are updating the state and trying to read and return the value in the same function which I think will never work due to async behaviour of state. (this is why your this.state.response in the return statement is the previous value, not the updated state value).
I'm not sure but you can write a callback on this.setState and return from there, the callback will read the updated state value and you can return the new state.

ReactJS Front-end with CodeIgniter backend : How build a Login System?

I'm an jQuery/Angular Developer last 5 years, and now I've planned move to ReactJS. I'm learning ReactJS by my own, as comparing the similar stuff what I did in jQuery (or Angular). My plan is, just to make a Login-Logout App, with ReactJS front-end and CodeIgniter - MySQL are the backend and database, respectively. Also I know what is CORS, and I'm pretty clear about the REST concepts. Let me explain my react app in detail.
My App component, calls an API via fetch, and upon the response which tells either session is set (as user is logged in) or session isn't set (user logged out/session expired), then the component will render either the 'Login Panel' component or a 'Dashboard' component. It works fine, as I tested. This is my App component.
class App extends Component {
constructor(props) {
super(props); //compulsary
this.state = { isUserAuthenticated: false };
}
componentDidMount() {
var innerThis=this;
fetch('http://localhost/forReact/index.php/general/authenticate',{
method:'GET',
Content_type:'application/json',
headers: {'Accept': 'application/json', 'Content-Type': 'application/json'},
mode: 'cors',
cache: 'no-cache'})
.then(res => { return res.json() })
.then(data => { alert(data.message); this.setState({isUserAuthenticated:data.result}); })
.catch(err => console.error(err));
}
render() {
return (this.state.isUserAuthenticated?<Home/>:<First/>);
}
}
export default App;
This is my CodeIgniter Controller's relevant method (the controller name is General.php)
public function authenticate(){
$session_data = $this->session->get_userdata();
if (is_null($session_data)) {
$this->sendJson(array("message"=>"No session","result"=>false));
}
else if (empty($session_data['username'])) {
$this->sendJson(array("message"=>"No username index","result"=>false));
}
else if ($session_data['username']=="") {
$this->sendJson(array("message"=>"Empty Username","result"=>false));
}
else{
$this->sendJson(array("message"=>"Valid Session","result"=>true));
}
}
This is the login component, which is being rendered in the App component, when there are no sessions set.
class LoginPanel extends React.Component {
constructor(props) {
super(props); //compulsary
this.unRef = React.createRef();
this.pwRef = React.createRef();
this.handleSubmit = this.handleSubmit.bind(this);
}
handleSubmit(){
var toServer={"userName":this.unRef.current.value,"passWord":this.pwRef.current.value};
fetch('http://localhost/forReact/index.php/general/login',{
method:'POST',
Content_type:'application/json',
headers: {'Accept': 'application/json', 'Content-Type': 'application/json'},
body: JSON.stringify(toServer),
mode: 'cors',
cache: 'no-cache'})
.then(res => { return res.json() })
.then(data => { alert(data.message); })
.catch(err => console.log(err));
}
render() {
return (
<form onSubmit={this.handleSubmit}>
<input ref={this.unRef} type="text" placeholder="Username"/>
<input ref={this.pwRef} type="password" placeholder="Password"/>
<button style={{margin:'1%'}} type="submit">Login</button>
<button style={{margin:'1%'}} type="reset">Help</button>
</form>);
}
}
export default LoginPanel;
Finally, this is the CodeIgniter Controller's relevant method (the same controller General.php)
public function login(){
$arrived = json_decode(file_get_contents('php://input'), true);
if (!empty($arrived['userName'])&&!empty($arrived['passWord'])) {
$fromDB=$this->Mdl_general->login($arrived['userName'],$arrived['passWord']);
if ($fromDB['result']) {
$this->session->set_userdata('username',$arrived['userName']);
$this->sendJson(array("message"=>$fromDB['message'],"result"=>$fromDB['result']));
}
else{
$this->sendJson(array("message"=>"Username and or Password is/are incorrect!","result"=>false));
}
}
else{
$this->sendJson(array("message"=>"Invalid or Missing Input Parameters!","result"=>false));
}
}
My issue is, when I logged in with the correct username and password (what I have in the mysql table), request was successfull and the response alert is showing; However, it seems the session is not set by the below php code,
$this->session->set_userdata('username',$arrived['userName']);
Whereas, if I did the above 'set session' code via a separate controller function, it sets the session successfully. However, even after that, the App component's API Call, still getting the response as the session is not yet set.
What might be the reason? I tried this via jQuery's Ajax Call and it works. I knew the fetch is much more different from the Ajax Call of jQuery. Could anyone explain, why this login component, didn't trig the PHP code to set the session?
That's not how it works. I will cut it to the short. The thing that you are trying to implement is not possible. In a very simple term, you are dealing with 2 different components now, your front end and the backend. And to communicate, you are using HTTP protocol as it is stateless every new request won't know anything about the previous one. In that case we need to reauthenticate every request. Check this comment to understand how to authenticate.
EDIT:
Just in case if you want to use REST
check this out

whats the proper way to make dependent api call in react redux

In my application, we maintain an inventory of authors and books and the vendors who are selling those books.
In my react js app I have a home page which displays a table with a list of authors and when we click on any author we go to author details page and URL will be like ${url}/author/12 . This author details page consists of three sections .First div contains author details like name, city, country.Second div contains list of books author has published so far which is a table and by default first record is selected.Third div contains the details of each book like title, description, price when a book is selected from second div .
In this author details page i use the id from props.match.params.id and make first api call to fetch author details, if success then fetch books if that is success get selected book details. I am confused between two approaches i have.
Method 1:
Make all the api calls from ui component based on data recieved. Make use of componentwillReceiveProps to know when the author details are fetched or books are fetched. Is componentwillReceiveProps the right place to decide to make subsequent api calls ?
class AuthorPage extends React.Component {
state = {
authorId:this.props.match.params.id,
}
componentDidMount() {
this.props(fetchAuthorDetails(authorId));
}
componentWillReceiveProps(newProps){
//Is this the right place to decide the subsequent api calls ?
if(isNotEmpty(newProps.author.details))
{
this.props(fetchAuthorBooks(authorId));
}
else if (newProps.author.books.length > 0 && isNotEmpty(newProps.author.selectedBook)){
this.props(fetchSelectedBookDetails(newProps.author.selectedBook.id));
}
}
render() {
// UI render code goess here
}
}
const mapDispatchToProps = dispatch => bindActionCreators(
{
fetchAuthorDetails,
fetchAuthorBooks,
fetchSelectedBookDetails
},
dispatch,
)
const mapStateToProps = (state, ownProps) => ({
authorDetails: state.author.details,
books: state.author.books,
selectedBook: state.author.selectedBook,
selectedBookDetails : state.author.selectedBook.details
});
export default connect(mapStateToProps)(AuthorPage);
Method2:
Is it better to make Subsequent dispatch calls (from ActionCreators dispatch calls depending on success/fail) like I am calling authorDetailsSuccess /BooksActionCreators.getAuthorBookDetails and authorDetailsError/ErrorsActionsCreator in the below code? And should i have separate action creators file for authors and books.
export function fetchAuthorDetails(authorId) {
dispatch(authorDetailsStart());
return dispatch =>
fetch('http://localhost/api', {
method: 'get',
headers: {
'Accept': 'application/json',
'Content-Type': 'application/json',
},
body: JSON.stringify({
email: userData.email,
password: userData.password,
}),
})
.then(response => {
if (response.status >= 200 && response.status < 300) {
console.log(response);
dispatch(authorDetailsSuccess(authorDetails));
dispatch(BooksActionCrerators.getAuthorBookDetails(authorDetails.id)); //is this the right place to chain
} else {
const error = new Error(response.statusText);
error.response = response;
dispatch(authorDetailsError());
dispatch(ErrorsActionsCreator.sendError(error));
throw error;
}
})
.catch(error => { console.log('request failed', error); });
}
Personally I always add redux-thunk module to my projects. This module transform a simple redux action to a promise-like function. So I can concatenate multiple actions by using classic .then() sintax. Check documentation for more details.

How to handle the case where the axios response comes after the component is rendered?

I am creating a blog application in rest framework and reactjs. On the home page, under componentDidMount, I send an API call using axios to get all the articles and setState of articles to the return. As I have studied, axios works on the idea of promise such that the code doesnt proceed, if the API is not fetched for a particular component. Please tell me, if I am wrong.
Then, I send a GET call to get the writer's name, who wrote the article by the id. Though, I assumed that the axios works as a promise. But, it doesnt work that way. Now, I am not sure how to move ahead.
Here is a snippet. So, in mainBody.js, I make the api call as:
class MainBody extends Component {
state = {};
componentDidMount () {
this.get_all_articles();
};
get_writer_name (id) {
let authstr = 'Bearer ' + window.localStorage.token;
let writer_url = "http://localhost:8000/api/writer/" + id.toString() + "/";
axios.get(writer_url, { headers: { Authorization: authstr }})
.then(response => {
console.log(response.data['name'])
return response.data['name'];
})
.catch(err => {
console.log("Got error")
})
};
get_all_articles () {
let authstr = 'Bearer ' + window.localStorage.token;
axios.get("http://localhost:8000/api/articles/", { headers: { Authorization: authstr }})
.then(response => {
this.setState({articles: response.data});
})
.catch(err => {
console.log("Got error")
})
}
render () {
return (
{this.state.articles.map((article, key) =>
<ArticleView key={article.id} article={article} writer_name={this.get_writer_name(article.created_by)} />
)}
)
}
}
In articleview2, I print all the data that is present in each of the articles along with the writer's name.
My articleview class is:
class ArticleView extends Component {
state = {article: this.props.article};
componentDidMount() {
console.log(this.props.writer_name;
}
render () {
return (
<React.Fragment>
<h2>{article.title}</h2>
<p>{article.body}</p>
<span>{this.props.writer_name}</span>
</React.Fragment>
)
}
}
If you see closely, I wrote two console.log statements to get the writer names. Based on the order, first the console log present in articleview class runs, which is undefined, and thenafter the data is fetched from the API call and the console log runs which returns the correct writer name.
I wanted to know, where is the error? Also, as I noticed, there are too many API calls being made to get the writer's name multiple time for all the listed articles. What are the industry best practices for these cases?
I want to know where is the error.
When you are writing this.state.articles.map(), means you're using property map of the Array articles which may be undefined before the data is fetched that will cause you the error Cannot read property map of undefined.
Solution
Now, as the API request is asynchronous, means render method will not wait for the data to come. So what you can do is use a loader variable in the state, and set it to true as long as the request is being made, and when the response has come, make it false, and show the loader in render when this.state.loader is true, and show articles when it is false.
Or you can initialize this.state.articles with an empty array that won't cause you the error.
Also, as I noticed, there are too many API calls being made to get the writer's name multiple time for all the listed articles. What are the industry best practices for these cases?
It is extremely bad practice to make an API request in the loop. Even myself has been scolded on it once I did it in my company.
Solution
You have tell your backend engineer to provide you filter for including the writer's name in each object of the article. We use Loopback on our backend, which provides a filter for including the related model in each object internally.
Since your API calls have a lot of things in common, you should first set up an axios instance that re-uses those common features:
const api = axios.create({
baseURL: 'http://localhost:8000/api/',
headers: { Authorization: `Bearer ${localStorage.token}` }
});
Now since your MainBody needs to fetch the resources from the API asynchronously, there will be a short period where the data is not yet available. There are two ways you can handle this. Either the MainBody can be responsible for making all the calls, or it can be responsible for just making the call to get all the articles, then each of the ArticleView components can be responsible for getting the writer's name. I'll demonstrate the first approach below:
class MainBody extends Component {
state = { articles: null, error: null, isLoading: true };
async componentDidMount () {
try {
const response = await api.get('articles/');
const articles = await Promise.all(
response.data.map(async article => {
const response = await api.get(`writer/${article.created_by}/`);
return { ...article, writer_name: response.data.name };
})
);
this.setState({ articles, isLoading: false });
} catch (error) {
this.setState({ error, isLoading: false });
}
}
render () {
const { articles, error, isLoading } = this.state;
return isLoading ? 'Loading...' : error
? `Error ${error.message}`
: articles.map(article => (
<ArticleView
key={article.id}
article={article}
writer_name={article.writer_name}
/>
)
);
}
}

Proper way to use fetch with github api v3?

I want to map all the issue titles for a repo and render them to li in a react component. I've been able to extract only the issue title using octokat library but unfortunately that library does not play nice with some other stuff i've got going on. Also, I'm sure i can do this with es6 and the fetch api.
Original Post:
Been having a hellava time finding info on using es6 fetch / promises with the github api. I'm trying to fetch issues for a repo but i'm obviously missing something... I've used a few different libraries to try and accomplish this but I'd like to just use fetch.
Update: Now my new goal is merely to console.log the titles... baby steps.
import React from "react";
export default class extends React.Component {
constructor() {
super()
this.state = {
issues: []
}
}
componentWillMount() {
// let issueArry = []
const url = "https://api.github.com/repos/fightplights/chode-stream/issues"
fetch(url)
.then(response => response) // i've tried with and without .json()
.then(body => console.log(body))
// this logs the same response
}
render() {
return (
<div>
<ul>
{here i will map this.state.issues (once i can get them =))}
</ul>
</div>
)
}
}
I know it's a simple error on my part. I'm still a fledgling. =)
...and obviously i need to render to the dom (not shown here)
#Andy_D is right that I wasn't parsing the body of the response, only the response itself (doh!) however now that i'm trying to data.map(issue => ...) I still get the error that data.map() is not a function. This leads me to believe that the data object is not the object the droids are looking for... hand waves in front of screen
Here's the response object in the first promise:
Response {
type: "cors",
url: "https://api.github.com/repos/fightp…",
redirected: false,
status: 200,
ok: true,
statusText: "OK",
headers: Headers,
bodyUsed: false
}
But when I take that response in the next
.then(data => console.log(data))
It is undefined...
I have tried returning the first response => response.json() (as andy suggested) but it's already json... am i wrong?
You need to read the JSON from the body of the response, what you are attempting to render is the full response body.
So:
fetch(url)
.then(response => response.json())
.then(data => data.map(...)
response.json() returns a promise, so you can .then it, the return value (data) will be the JS object/array you're looking for.
https://developer.mozilla.org/en-US/docs/Web/API/Body/json

Resources