Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Review the "terms" message on library loading. Sufficient? #27

Closed
Tracked by #26
mari-williams opened this issue Dec 10, 2022 · 3 comments
Closed
Tracked by #26

Review the "terms" message on library loading. Sufficient? #27

mari-williams opened this issue Dec 10, 2022 · 3 comments
Milestone

Comments

@mari-williams
Copy link
Contributor

mari-williams commented Dec 10, 2022

Can we 'insert' this somewhere in the beginning and have an 'accept' box to click before proceeding? I really don't know what I'm asking for in terms of code, just looking to set up a small stumbling block where folks see this before they download data - as they would via the front end.

@eeholmes
Copy link
Member

eeholmes commented Dec 13, 2022

@mari-williams I added a start up message when you first type library(rCAX)

Access and use of data in the CAX databases requires acceptance of the terms of use: see rcax_termsofuse()

eeholmes added a commit that referenced this issue Dec 13, 2022
@eeholmes eeholmes added this to the v1.0.0 milestone Dec 14, 2022
@eeholmes eeholmes changed the title https://nwfsc-math-bio.github.io/rCAX/articles/terms.html This is from the Excel files. Anything missing that should be said? Review the "terms" message on library loading. Sufficient? Dec 14, 2022
@eeholmes
Copy link
Member

Alternatively, I could actually have a Y/N accept terms of use message and require the user to say Y. Might get irritating...

@mari-williams
Copy link
Contributor Author

Let's keep that in the back of our minds but not implement it while we are futzing around.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants