learningequality / learningequality/studio

Post-release code cleanup of some metadata inheritance code

Open
#4,693 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

P1 - important TAG: dev experience TAG: tech update / debt
Dominant language
Python
Stars
191
Forks
307
Avg merge
5d 6h
Merged PRs (30d)
10

Description

Observed behavior

As discussed in the PR for MVP implementation of the metadata inheritance modal, some of the code is quite long, and the readability could be improved.

Dev-facing consequences

It could be harder to maintain this code or spot bugs. May be trickier for others to contribute in this area with enhancements or bug fixes.

Context

Studio Bulk Editing release - unstable as of August 30 2024

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

The issue names no file or test; start with the metadata inheritance modal implementation and review discussion in PR #4664. Identify the code paths whose length or readability needs attention, agree on the cleanup scope with maintainers, and confirm the refactor preserves existing behavior.

Written by the indexing model from the issue text.

Assessment

Domain
developer-experience
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.