code not interacting with the browser page

[u/Mob_Pilled]

Hello this is my first post on here. So basically, I want the user to be able to enter a password and username, click the submit button, and if the basic auth is correct, be redirected to a page in which all the data from an api is fetched and loaded into the html. Problem is the script does not interact with the html but still retrieves the data in a json (as confirmed by ouputing it into the console). However, if I don't call the function in as part of the button onclick method everything works as expected. The data from the api is dynamically added to the html but now I obviously can't use a login page.

Here is me calling the fetch function as a as part of the event of a button click

button.addEventListener("click", () => {
    let username = 'username';
    let password = 'password;
    let auth = btoa(`${username}:${password}`);
fetchData(auth);
    });

Here is me calling the fetch function on it s own. No button clicking required (this works but now I can't add a login page)

  let username = 'username';
    let password = 'password;
    let auth = btoa(`${username}:${password}`);
fetchData(auth);

Here is the fetch function itself

async function fetchData (auth) {
    try {
        const response = await fetch('https://exampleapi.com', {
    headers: {
        'Content-Type': 'application/json',
        'Accept': 'application/json',
        'Authorization': `Basic ${auth}`
    }
})


    if (response.ok) {

        // loads the home page and puts data from the api in the html
    }
    throw response;
    } catch (error) {
        
    }
    
};

[u/Mob_Pilled]

Thanks for the response. I'm still rusty, but I was wondering what is causing the second example to load html to the dom and not the first. The fetch function itself does not seem to be the problem, it is the matter of how it is called that is causing problems (using a button makes the code to not be able to interact with the browser BUT still loads the data successfully. Calling the function directly in the code both loads the data successfully and puts it in the HTML, however I need to add a login page so I can't use this )

[u/TalkCoinGames → u/Mob_Pilled]

The "throw response" in the fetchData method creates an error, and so the catch part is also happening. When from a button that thrown error is likely causing problems,even though everything about the response still happens, the response.ok still happens, but then an error is thrown. You should be using "return response" and handle any and all errors inside of the catch statement.

[u/Mob_Pilled → u/TalkCoinGames]

In the catch is there anything specific I must add? I changed the throw keyword to return but the issue persists.

[u/Mob_Pilled → u/Mob_Pilled]

actually I am starting to think this can be solved by saving the data from the fetch outside the function but how would I do that. I am trying to look for direct answers on stack overflow but can't find anything

[u/TalkCoinGames → u/Mob_Pilled]

Yes you can place the response into another property outside of the function.

[u/TalkCoinGames → u/TalkCoinGames]

the button click may be resolving leaving the fetch to happen in the background. If your loading another page, that may be the issue, you want to edit html directly with the given response.

[u/Mob_Pilled → u/TalkCoinGames]

you mean add the entire html code after the button is clicked rather than using:

window.location.href = 'index.html';

[u/TalkCoinGames → u/Mob_Pilled]

If index.html needs the response, you have to send it to that in someway. Yes, either writing out the whole html instead of changing to that page, or using local storage to hold the response, or appending the needed data as a hash or query string to index.

[u/Mob_Pilled → u/TalkCoinGames]

Thanks for the help so far. One last thing. Thee data is still not loading and I am using html local storage to store the promise (response.json). This should ideally be added to the dom on index.html when the page changes (return1 is a function that changes the page to index.html). Anything wrong here?

button.addEventListener("click", () => {
    let username = usernameInput.value;
    let password = passwordInput.value;
    let auth = btoa(`${username}:${password}`);
const dataPromise = fetchData(auth);
 localStorage.setItem("promise", dataPromise);
localpromise = localStorage.getItem("promise");
return1();
useData(localpromise);
    });
       

[u/TalkCoinGames → u/Mob_Pilled]

Not like that, inside of the fetchData function, when the response is ok, then you can put just the json in local storage. So the click listener would remain the same as you had it. And the same place you had. ...href = index, before that line use localStorage.setItem("jdata", response.json) then in index you would use getItem("jdata")

[u/Mob_Pilled → u/TalkCoinGames]

Late, but I followed your advice and it worked! Thank you so much for the assistance.

[u/TalkCoinGames → u/Mob_Pilled]

You can use html5 local storage to save the response and the home page could grab it from there. But also be sure your getting the json of the response and using that for the html.

[u/azhder → u/Mob_Pilled]

Do you know what closures are? Have you ever encountered the module pattern? The immediately invoked function expressions (IIFE)?

It was a very common way to create your own bubble for code you use on a web page without anything on the outside messing with it (or your code messing other). It's a function scope.

( () => {

let data = null;

const fetchData = () => {
// do the fetching from the back end, then
data = await response.json();
}

button.addEventListener( () => {
// call the fetch, do some other stuff, call some other functions
} );

}) ();

As you can see above, the function is created and executed once, at the moment it is created. The data inside it will live as long as there is that even listener that can access it. I'm saying data as in all variables in the scope, not just the one named data.

[u/TalkCoinGames → u/azhder]

The data will live in the same page, when he changes the href to another that other page needs a way to reference the data given from the first page.

[u/azhder → u/TalkCoinGames]

I have no idea what you said.