Greenstand / Greenstand/treetracker-web-map-core

Migrate event system to use node events lib.

Open
#22 3 comments 0 reactions 0 assignees View on GitHub
enhancement good first issue high-priority
Dominant language
JavaScript
Stars
4
Forks
33
PR merge metrics
No merged PRs in 30d

Description

We impletement the event system by simply call onXXX function parten, in this PR https://github.com/Greenstand/treetracker-web-map-core/pull/20 https://github.com/Greenstand/treetracker-web-map-core/pull/21 we try to use `events` lib to provide a solid and battle checked approach to implement event system, please migrate all other onXXX event to use
These listner is like onError, onLoad...

---

Some hints:

- Please read our [readme](https://github.com/Greenstand/treetracker-web-map-core#treetracker-web-map-core) for more information/guide/tutorial.
- Here is [an engineering book](https://greenstand.gitbook.io/engineering/) in Greenstand.
- To know more about our organization, visit our [website](https://greenstand.org).
- If you want to join the slack community (some resources need the community member's permission), please leave your email address.

Contributor guide

No contributing guide indexed for this repository

Research direction

Read the README and the approaches in pull requests 20 and 21 first. Search the repository for the remaining onXXX listeners, including onError and onLoad, and identify the existing event-system entry points. Done means all remaining onXXX event handling uses the events library consistently.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, node.js
Domain
frontend
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.