DocumentPicker.isCancel not working

[manuelwenner]

Hello together, can someone tell me how to use .isCancel the right way. For me it didn't work out like this. Can someone please help?

    try {
        DocumentPicker.pick({
            type: types.pdf,
        })
        .then(setResult) 
        } catch (err) {
            setLoading(false)
        if (DocumentPicker.isCancel(err)) {
            // User cancelled the picker, exit any dialogs or menus and move on
            console.log("cancel");
            setLoading(false)
        } else {
            throw err;
        }
    }

[vonovak]

hello Manuel, thanks for asking

First, let me reformat it:

try {
  DocumentPicker.pick({
    type: types.pdf,
  }).then(setResult);
} catch (err) {
  setLoading(false);
  if (DocumentPicker.isCancel(err)) {
    // User cancelled the picker, exit any dialogs or menus and move on
    console.log("cancel");
    setLoading(false);
  } else {
    throw err;
  }
}

now with try and catch aligned, it's easier to see that you're combining the try + catch syntax with Promise.then syntax

I suggest you rewrite it as

try {
  const result = await DocumentPicker.pick({
    type: types.pdf,
  })
  setResult(result)
} catch (err) {
  setLoading(false);
  if (DocumentPicker.isCancel(err)) {
    // User cancelled the picker, exit any dialogs or menus and move on
    console.log("cancel");
    setLoading(false);
  } else {
    throw err;
  }
}

the important part is awaiting the promise.

hope this helps, thank you :)

[manuelwenner]

Thanks @vonovak for your answer and your time. It worked out without awaiting the promise and using .isCancel. Here is what works fine for me:

setLoading(true);
DocumentPicker.pick({
    type: types.pdf,
})
.then(setResult) 
.catch(() => {
    setLoading(false);
});    

[vonovak]

Hi, just to clarify, your end result is semantically the same as mine, just different syntax. You're awaiting the promise as well, just not using the await keyword 👍