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.

Sign up for GitHub

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

Jump to bottom

proofr v0.3.0 Updates #31

Merged
merged 29 commits into from
Sep 16, 2024
Merged

proofr v0.3.0 Updates #31

merged 29 commits into from
Sep 16, 2024

Conversation

tefirman
Copy link
Member

@tefirman tefirman commented Sep 6, 2024 •
edited
Loading

Description

Related Issues

Testing & Examples

  • See links to individual PR's listed above

tefirman and others added 23 commits May 31, 2024 15:34
@tefirman
Merge pull request #17 from getwilds/main
b97e9b9 
Catching dev branch up with main
@sckott
rework internals to use httr2 and dev version of webmockr that suppor…
e98ba31 
…ts httr2
@sckott
update docs
7171223
@sckott
Merge branch 'main' into httr2-webmockr
d8afed7
@dtenenba @sckott
create proof_info() function to call /info endpoint
80ce107
@dtenenba @sckott
add man page
84a2172
@dtenenba @sckott
add unit test for info endpoint
011ddff
@tefirman @sckott
Removing logged_in_user and api_endpoint from info function
04d3c9b
@sckott
merge from dev
8a9cc17
@sckott
no longer need dev version of webmockr; tweak url formatting in desc …
ce8f9c3 
…file
@sckott
swap info fxn over to use httr2
0d8f6ea
@sckott
R 4.1.0 or greater for native pipe
f64a28c
@sckott
Merge pull request #25 from getwilds/httr2-webmockr
5de3e90 
Swap httr out for httr2
@sckott
improve docs for proof_authenticate, pushing users towards more secur…
974a122 
…e use of the fxn
@sckott
fix #13 remove links to proof api repo since its private - levaing re…
5a32a97 
…ference in readme though
@sckott
fix #22 fix install line for rcromwell
e8212c3
@sckott
fix #24 document ability to set api base url via an en var in pkg lev…
9074372 
…el doc and readme
@sckott
fix #23 add advice to vignette on storing proof token in .Renviron file
58419fa
@tefirman
Merge pull request #29 from getwilds/rcromwell-install-fix
6ef2c03 
fix install line for rcromwell
@tefirman
Merge pull request #30 from getwilds/env-vars
6b82286 
Documentation improvements around env vars
@tefirman
Merge pull request #28 from getwilds/remove-proof-api-links
4724f63 
remove links to proof api repo since its private
@tefirman
Merge branch 'dev' into better-auth-docs
11337b9
@tefirman
Merge pull request #27 from getwilds/better-auth-docs
153ec51 
improve docs for proof_authenticate, pushing users towards more secure practices
@tefirman tefirman changed the title proofr v0.3.0 proofr v0.3.0 Updates Sep 6, 2024
@tefirman
Copy link
Member Author

@sckott -- Could you take a look at the linting errors identified in the GitHub Actions? They're pretty minor, so I'm fine just adding nolint if that's easier.

@sckott
Copy link
Member

sckott commented Sep 12, 2024

I'll check it out

@sckott
Copy link
Member

sckott commented Sep 12, 2024

12a49b6 should fix that issue, we'll see

sckott
sckott approved these changes Sep 16, 2024
View reviewed changes
Copy link
Member

@sckott sckott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ran the shiny app locally with this dev version and did not see any issues

@tefirman tefirman marked this pull request as ready for review September 16, 2024 22:27
@tefirman tefirman merged commit 51a2c43 into main Sep 16, 2024
8 checks passed
tefirman added a commit that referenced this pull request Sep 16, 2024
@tefirman
Merge pull request #34 from getwilds/main
4cdd2f6 
Merge pull request #31 from getwilds/dev
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sckott sckott sckott approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@tefirman @sckott @dtenenba