Open leonardosfl opened 4 years ago
preprintId should be used in the docId attribute, just like here: https://github.com/prereview/rapid-prereview/blob/16e9bfe7ca522edab2bfb4f85b855d37b33313a5/src/components/extension-fallback.js#L67-L68
preprintId
docId
This was causing an error since the preprintId parameter was incorrect.
I also noticed that the fallback PDF is always being fetched, even if it's not used. Not sure if this is a bug.
According to this line of code, it should be a fallback, right? https://github.com/prereview/rapid-prereview/blob/16e9bfe7ca522edab2bfb4f85b855d37b33313a5/src/components/extension-fallback.js#L79
preprintId
should be used in thedocId
attribute, just like here: https://github.com/prereview/rapid-prereview/blob/16e9bfe7ca522edab2bfb4f85b855d37b33313a5/src/components/extension-fallback.js#L67-L68This was causing an error since the
preprintId
parameter was incorrect.I also noticed that the fallback PDF is always being fetched, even if it's not used. Not sure if this is a bug.
According to this line of code, it should be a fallback, right? https://github.com/prereview/rapid-prereview/blob/16e9bfe7ca522edab2bfb4f85b855d37b33313a5/src/components/extension-fallback.js#L79