Skip to content

Unclear Readme  #7

Description

@cbruce11

Hey, we found the readme to be a little confusing for the following reasons:
First, we couldn't seem to understand the location of the documents folder that was being referenced. In the end we found that using knownFolders.currentApp() instead of knownFolders.documents() put as at the root of the application structure. At that point we could create a documents folder as a sibling to App_Resources and store our files there and then access them.
Second, in your code sample you create a this.file variable and initialize it but then never use it. This led to a small mixup on our side which we should have caught on our own but it seems strange to include it anyway.

Also we noticed the repo hasn't been maintained for over a year and we were wondering if we could fork the repo and then just make and maintain our own.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions