placing the required node version in a .node-version file allows it to be read programatically by node version managers like nodenv, asdf, etc
3.5 KiB
Contributing to Unleash
Getting started
Before you begin:
- Have you read the code of conduct?
- Check out the existing issues
- Browse the developer-guide for tips on environment setup, running the tests, and running Unleash from source.
Don't see your issue? Open one
If you spot something new, open an issue. We'll use the issue to have a conversation about the problem you want to fix.
Ready to make a change? Fork the repo
Fork using GitHub Desktop:
- Getting started with GitHub Desktop will guide you through setting up Desktop.
- Once Desktop is set up, you can use it to fork the repo!
Fork using the command line:
- Fork the repo so that you can make your changes without affecting the original project until you're ready to merge them.
Fork with GitHub Codespaces:
- Fork, edit, and preview using GitHub Codespaces without having to install and run the project locally.
Make your update:
Make your changes to the file(s) you'd like to update. You'll need Node.js v14 and PostgreSQL 10 to run Unleash locally. See more details
Open a pull request
When you're done making changes and you'd like to propose them for review by opening a pull request.
Submit your PR & get it reviewed
- Once you submit your PR, others from the Unleash community will review it with you. The first thing you're going to want to do is a self review.
- After that, we may have questions, check back on your PR to keep up with the conversation.
- Did you have an issue, like a merge conflict? Check out GitHub's git tutorial on how to resolve merge conflicts and other issues.
Your PR is merged!
Congratulations! The whole Unleash community thanks you. ✨
Once your PR is merged, you will be proudly listed as a contributor in the contributor chart.
Nice to know
Controllers
In order to handle HTTP requests we have an abstraction called Controller. If you want to introduce a new route handler for a specific path (and sub pats) you should implement a controller class which extends the base Controller. An example to follow is the routes/admin-api/feature.ts implementation.
The controller takes care of the following:
- try/catch RequestHandler method
- error handling with proper response code if they fail
await
the RequestHandler method if it returns a promise (so you don't have to)- access control so that you can just list the required permission for a RequestHandler and the base Controller will make sure the user have these permissions.