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

Create read_wi.R #145

Merged
merged 58 commits into from
Sep 6, 2022
Merged

Create read_wi.R #145

merged 58 commits into from
Sep 6, 2022

Conversation

Rafnuss
Copy link
Collaborator

@Rafnuss Rafnuss commented Aug 23, 2022

No description provided.

@Rafnuss Rafnuss mentioned this pull request Aug 23, 2022
@peterdesmet
Copy link
Member

@Rafnuss can we assume that deploymentID is unique across projects?

@Rafnuss
Copy link
Collaborator Author

Rafnuss commented Aug 23, 2022

DeployementID is manually defined by the user. I could see a case where a user would use the same deployementID for two different project. But as we are only authorising a single project at the time, I don't think this would be an issue. Or did I misunderstood your question?

Move locationID above locationName
Set camera_id to character
Correct favourite
Enable sci_name
And correct code to handle resources
@peterdesmet peterdesmet self-requested a review August 23, 2022 17:39
R/read_wi.R Outdated Show resolved Hide resolved
R/read_wi.R Outdated Show resolved Hide resolved
R/read_wi.R Outdated Show resolved Hide resolved
R/read_wi.R Outdated Show resolved Hide resolved
R/read_wi.R Outdated Show resolved Hide resolved
R/read_wi.R Outdated Show resolved Hide resolved
R/read_wi.R Outdated Show resolved Hide resolved
R/read_wi.R Outdated Show resolved Hide resolved
R/read_wi.R Outdated Show resolved Hide resolved
R/read_wi.R Outdated Show resolved Hide resolved
R/read_wi.R Outdated Show resolved Hide resolved
@peterdesmet peterdesmet self-requested a review August 30, 2022 09:43
R/read_wi.R Outdated Show resolved Hide resolved
@peterdesmet peterdesmet merged commit 5fa091b into inbo:main Sep 6, 2022
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.

3 participants