parse-community / parse-community/parse-server
Parse Server options should set default
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
- I am not disclosing a vulnerability.
- I am not just asking a question.
- I have searched through existing issues.
Current Limitation
Currently, to add Parse Server options, developers have to add validation for the variable type, as well as set the default. The error messages for if a variable doesn’t enforce a type is inconsistent. And, you cannot set environment variables to deep Parse Server options. Overall, it needs some TLC.
Feature / Enhancement Description
All this information is already typed in the definitions, so the type and default should be automatically set. The only custom validation should be for stricter rules, such as checking that a parameter is greater than 0, for example.
Example Use Case
Creating a new server option would require much less code if the type handling and defaults were handled internally.
Alternatives / Workarounds
Related: #7151
3rd Party References
n/a
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 file, test, or entry point is named in the issue. Start by reviewing the existing Parse Server option definitions and related issue #7151; done would mean automatic type and default handling, consistent validation errors, and support for environment variables on nested options.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, node.js
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100