Skip to content
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

Rails budget app #2

Merged
merged 22 commits into from
Aug 14, 2022
Merged

Rails budget app #2

merged 22 commits into from
Aug 14, 2022

Conversation

deyemiobaa
Copy link
Owner

@deyemiobaa deyemiobaa commented Aug 12, 2022

Hello reviewer. Thanks for taking out time to view this PR. Here's a rundown of the milestones completed for this project.

  • Display a splash screen for new users visiting the app. It leads to a sign-in and login page
  • Users can sign up with their name, email, and password
  • Users can create a category and upload an icon for it
  • Users can create payments/transactions for each category
  • The app is styled according to the template provided
  • Unit and integrations have been written and asserted to confirm the behavior of the app matches the project expectations

Live link available here
Video presentation available here

Copy link

@RolandM99 RolandM99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Status: Approve 🥇 ✔

Hello @deyemiobaa,

For now your project is complete! There is nothing more important else to say other than... it's time to merge your PR :shipit:

KUDOS! 🎉

Cheers and Happy coding!👏👏👏

  • [OPTIONAL] I'll just suggest you add a screenshot of your web and mobile version of your app in your Readme file.
    Feel free to leave any questions or comments in the PR thread if something is not 100% clear.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

@deyemiobaa deyemiobaa merged commit a5b127e into main Aug 14, 2022
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