liquid-labs / liquid-labs/catalyst-core-api

After re-integrating the 'events', remove comment code.

Open
#6 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

js/core/resources/actions.js and js/core/resources/reducer.js deal with special casing 'events'. With the new 'routes' processing rules (though to be honest, I don't remember the change that enables this) we should be able to drop the special casing.

Until then, the code will be maintained just in case there's something helpful in there.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading the special-case handling for 'events' in js/core/resources/actions.js and js/core/resources/reducer.js, then compare it with the newer 'routes' processing rules. Confirm that the events behavior is covered by those rules before removing the special casing; done means both files no longer require the obsolete events-specific code.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.