Thanks for checking us out!
If you're new here, here are some things you should know:
- We actively curate issues and try to make them as self-contained as possible for people new to the application; those ones are tagged "Hacktoberfest" + "Help Wanted"
- We're actively watching for Pull Requests and you shouldn't have to wait very long for a review. Try to make sure your build passes (
rubocop -a
is a frequent need) and that you've addressed the requirements in the issue - There is a wiki article called Application Overview. It needs a tiny bit of updating, but is mostly still accurate. It will introduce you to some vocabulary and general concepts, if you find something confusing and want to find the answer on your own.
- Check the
CONTRIBUTING.md
file for a guide on how to get started - This is a 100% volunteer-supported project, please be patient with your correspondence. We do handle issues and PRs with more fervor during October, but most (all?) of us have day jobs and so responses to questions / pending PRs may not be immediate. Please be patient, we'll get to you! :)
Please feel free to join us on Slack! You can sign up at https://rubyforgood.org -- click on the Slack icon at the top to get an invite. We're in #diaper
The core team leads are: @armahillo @seanmarcia @mdworken @benreyn There are numerous other folks that can chime in and answer questions -- please ask and someone will probably be there to help!
This application is an inventory management system that is built to address the needs of Diaper Banks as directly and explicitly as possible. Diaper Banks maintain inventory, receive donations and other means of intaking diapers (and related supplies), and issue Distributions to community partner organizations. Like any non-profit, they also need to perform reports on this data, and have day-to-day operational information they need as well. This application aims to serve all those needs, as well as facilitate, wherever possible the general operations of the Diaper Bank themselves (eg. through using barcode readers, scale weighing, inventory audits).
For a general overview of the application, please see the Application Overview wiki article.
This project took what we built for the Portland Diaper Bank in 2016 and turned it into a multitenant application, something that all diaper banks can use. We re-used models, code and other documentation where applicable as well as implemented new features and functionality requested by the prime stakeholder (PDXDB). We're super excited to have had Rachel Alston, the director of the Portland Diaper Bank, attending our event in 2017, providing guidance and giving us the best chance of success!
The ubuntu_installation.md
file (https://github.com/rubyforgood/diaper/blob/master/ubuntu-installation.md) has detailed instructions for installation and configuration of an Ubuntu host to run this software. Although there is not a document for Mac OS, it may be helpful for that as well.
This app uses Ruby version 2.6.4, indicated in /.ruby-version
and Gemfile
, which will be auto-selected if you use a Ruby versioning manager like rvm
, rbenv
, or asdf
.
If you don't have Yarn installed, you can install with Homebrew on macOS brew install yarn
or visit https://yarnpkg.com/en/docs/install. Be sure to run yarn install
after installing Yarn. NOTE: It's possible that Node version 12 may cause you problems, see issue #751. Node 10 or 11 seem to be fine.
Be sure to create a .env
file in the root of the app that includes the following lines (change to whatever is appropriate for your system):
PG_USERNAME=username
PG_PASSWORD=password
If you're getting the error PG::ConnectionBad: fe_sendauth: no password supplied
, it's because you have probably not done this.
This app uses PostgreSQL for all environments. You'll also need to create the dev
and test
databases, the app is expecting them to be named diaper_dev
and diaper_test
, respectively. This should all be handled with rails db:setup
.
From the root of the app, run bundle exec rails db:seed
. This will create some initial data to use while testing the app and developing new features, including setting up the default user.
Run bundle exec rails s
and browse to http://localhost:3000/
To login to the web application, use these default credentials:
Organization Admin
Email: [email protected]
Password: password
User
Email: [email protected]
Password: password
Please feel free to contribute! While we welcome all contributions to this app, pull-requests that address outstanding Issues and have appropriate test coverage for them will be strongly prioritized. In particular, addressing issues that are tagged with the next milestone should be prioritized higher.
To contribute, do these things:
- Identify an issue you want to work on that is not currently assigned to anyone
- Assign it to yourself (so that no one else works on it while you are)
- (If not already a Contributor, fork the repo first)
- Checkout a new issue branch -- there's no absolute requirements on this, but we encourage the branch name format
XXX-brief-description-of-feature
whereXXX
is the issue number. - Do the work -- discuss any questions on the Issues as needed (we try to be pretty good about answering questions!)
- (If you created a new model, run
bundle exec annotate
from the root of the app) - Create tests to provide proof that your work fixes the Issue (if you need help with this, please reach out!)
- Commit locally, using descriptive commit messages that acknowledge, to the best of your ability, the parts of the app that are affected by the commit.
- Run the tests and make sure they run green; if they don't, fix whatever broke so that the tests pass
- Final commit if any tests had to be fixed
- Push up the branch
- Create a Pull Request - Please indicate which issue it addresses in your pull-request title.
Squashing your own commits before pushing is totally fine. Please don't squash other people's commits. (Everyone who contributes here deserves credit for their work! :) ). Also, consider the balance of "polluting the git log with commit messages" vs. "providing useful detail about the history of changes in the git log". If you have several (or many) smaller commits that all serve one purpose, and these can be squashed into a single commit whose message describes the thing, you're encouraged to squash.
There's no hard and fast rule here about this (for now), just use your best judgement.
At that point, someone will work with you on doing a code review (typically pretty minor unless it's a very significant PR). If TravisCI gives ๐ to the PR merging, we can then merge your code in; if your feature branch was in this main repository, the branch will be deleted after the PR is merged.
Try to keep your PRs limited to one particular issue and don't make changes that are out of scope for that issue. If you notice something that needs attention but is out-of-scope, please create a new issue.
Run all the tests with:
bundle exec rspec
This app uses RSpec, Capybara, and FactoryBot for testing. Make sure the tests run clean & green before submitting a Pull Request. If you are inexperienced in writing tests or get stuck on one, please reach out so one of us can help you. :)
The one situation where you probably don't need to write new tests is when simple re-stylings are done (ie. the page may look slightly different but the Test suite is unaffected by those changes).
Tip: If you need to skip a failing test, place pending("Reason you are skipping the test")
into the it
block rather than skipping with xit
. This will allow rspec to deliver the error message without causing the test suite to fail.
example:
it "works!" do
pending("Need to implement this")
expect(my_code).to be_valid
end
Sometimes we want to get a PR up there and going so that other people can review it or provide feedback, but maybe it's incomplete. This is OK, but if you do it, please tag your PR with in-progress
label so that we know not to review / merge it.
- The generated
schema.rb
file may include or omitid: :serial
forcreate table
, andnull: false
fort.datetime
. According to Aaron, this can safely be ignored, and it is probably best to commit the schema.rb only if you have committed anything that would change the DB schema (i.e. a migration). - If you have trouble relating to SSL libraries installing Ruby using
rvm
orrbenv
on a Mac, you may need to add a command line option to specify the location of the SSL libraries. Assuming you are usingbrew
, this will probably result in a command looking something like:
rvm install 2.6.4 --with-openssl-dir=`brew --prefix openssl`
.
Users that are frequent contributors and are involved in discussion (join the slack channel! :)) may be given direct Contributor access to the Repo so they can submit Pull Requests directly, instead of Forking first.
Thanks to Rachel (from PDX Diaperbank) for all of her insight, support, and assistance with this application, and Sarah ( http://www.sarahkasiske.com/ ) for her wonderful design and CSS work at Ruby For Good '17!