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

Replace API calls with {laminr.api} #38

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Replace API calls with {laminr.api} #38

wants to merge 7 commits into from

Conversation

lazappi
Copy link
Collaborator

@lazappi lazappi commented Oct 14, 2024

Use the {laminr.api} package for API calls.

Fixes #4

* origin/main:
  Add printing to remaining classes (#31)
  Improve error messages (#30)
  Update documentation (#29)
  Return `NULL` when a record's related field is empty (#28)
  minor fix in usage vignette (#32)
  Add usage vignette (#18)
  Add a simple unit test which queries laminlabs/lamindata (#27)
  bump action from v4.5.0 to v4 (#26)
* origin/main:
  Make InstanceAPI print methods public (#37)
@lazappi lazappi changed the title Replace API calls with **{laminr.api}** Replace API calls with {laminr.api} Oct 14, 2024
@rcannood
Copy link
Collaborator

Thanks for taking a look at this! With this, we can more easily start using other functionality from the API and keep up to date with changes.

Let's wait for this to be merged until after the 0.1.0 release, but other than that this looks pretty good to me!

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

Successfully merging this pull request may close these issues.

use laminr.api in API.R
2 participants