Jerricodelacruz/issue83 #88
Conversation
|
@jerricodelacruz - was this meant to be merged? |
I was not meant to combine it in main branch. I was just combining the issue 82 and 83 branch. |
|
|
||
| const handleSelect = (id : number) => { | ||
| setSelectedId(id); | ||
| } | ||
|
|
||
| const selectView = (view) => { | ||
| setSelectedView(view); |
There was a problem hiding this comment.
Are you using an off the shelf PDF viewer or creating one that can support other office document types?
There was a problem hiding this comment.
As of now we are still looking for the package for pdf viewer and other types of documents. Do you have any suggestion package that we can use?
Issue #81 Document viewer page - @dennisdelamida
Approved for merge @jerricodelacruz (feel free to merge yourself when approved, then delete the dangling branch) - some comments about the document viewer |
Combine Icon view and list view