donejs / donejs/bitcentive

models/alert.js tech debt

Open
#197 1 comment 0 reactions 0 assignees View on GitHub
cleanup
Dominant language
JavaScript
Stars
7
Forks
3
PR merge metrics
No merged PRs in 30d

Description

- [x] `alertClass` is UI related ... possibly move to helper.
- [x] `displayInterval` should use `Infinity` to represent never hiding.
- [ ] is an alert a model? Should this be something we move to some other project?

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading models/alert.js and the issue's completed notes about alertClass and displayInterval. Determine whether Alert is still an appropriate model or should move to another project, and document the resulting architectural decision; the issue does not name tests or another entry point.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.