parse-community / parse-community/parse-server
Remove `push_id` and `push_time` from push notifications
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 21.4k
- Forks
- 4.8k
- Avg merge
- 7h 45m
- Merged PRs (30d)
- 11
Description
New Feature / Enhancement Checklist
- Report security issues confidentially.
- Any contribution is under this license.
- Before posting search existing issues.
Current Limitation
See https://github.com/parse-community/parse-server-push-adapter/issues/237#issuecomment-2113336389; push_id and push_time seem to be legacy keys. Their purpose should be investigated. Maybe they can be removed.
Feature / Enhancement Description
Removal with care, as Parse Android SDK and developers' custom apps may use these keys for something, so they cannot be just removed but must be made optional for phasing out.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No source file, test, or entry point is named. Begin with the linked parse-server-push-adapter discussion and investigate the purpose and current consumers of push_id and push_time, including Parse Android SDK and custom apps. Done means an evidence-based compatibility plan for making the keys optional and a verified path for phasing them out.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- android, javascript, nodejs
- Domain
- api, backend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 32/100