posit-dev / posit-dev/connectapi

Update `set_environment_new()` function to use content `guid`

Open
#229 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
R
Stars
54
Forks
27
Avg merge
1d 3m
Merged PRs (30d)
1

Description

Currently set_environment_new() function takes an R6 class environment to update an environment variable on a particular piece of content. For ease of use, the input argument should be content guid and the function should generate the environment object as needed. The current behaviour adds an additional step to create an environment class object first, and it also does not match the behaviour of get_environment() function which takes content guid as input.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating set_environment_new() and compare its input handling with get_environment(), which already accepts a content guid. Update the function so callers provide the content guid and the environment object is created as needed; done means the extra R6 environment-object step is no longer required.

Written by the indexing model from the issue text.

Assessment

Tech stack
r
Domain
api
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.