Skip to content

Chad Jemmett Takehome project #113

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 42 commits into
base: master
Choose a base branch
from

Conversation

chadjemmett
Copy link

Hi Paul Chatterton, Kurt Gardner, and Kurt Christensen
Please review my Elixir take home project. Over the weekend I dove in and

  • fixed all the tests to accomodate a new company module.
  • I wrote a test to update the Company’s available_credit when a user makes a new transaction. Take a look here: New company test
  • Developed a function in the Company module to update the available_credit. You can see my work here: New company function

And please take a look at the additions I made earlier this year.

  1. Added Company Module
  2. Wrote seed file for testing DB

Chad Jemmett and others added 30 commits September 21, 2021 20:30
Fixed companies. Related company to transaction. seeded db
Can see all the models in the graphql
Chad Jemmett’s Application Project for a Role as Backend Dev at Divvy.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants