dandi / dandi/dandi-archive

Wrap rest functions with `computed`

Open
#744 0 comments 0 reactions 0 assignees View on GitHub
maintenance refactor zen task
Dominant language
Python
Stars
26
Forks
21
Avg merge
5d 6h
Merged PRs (30d)
14

Description

Probably out of scope for this PR, but we could probably just import `computed` in `rest.ts` and define these as `computed(() => ...)` instead of `() => ...`, which saves you having to do this song and dance in every component that needs to use them.

_Originally posted by @dchiquito in https://github.com/dandi/dandiarchive/pull/901#discussion_r717847282_

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating rest.ts and reading the functions referenced as “these,” then inspect the components that currently consume them. Confirm how the functions are used and whether existing tests cover them; done means the rest functions use computed as proposed and their consumers still work.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
frontend
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.