[DO-NOT-MERGE] Project review #86
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* [14][setup]add-files-structure
* [13][setup] install styled components and react icons libs
* inital style props base heade, contetnt * minor fix * . * buttons vunction implemented * fixed description * CR fixes-A * corrections, included proptypes * onCancel callback fixes * comments added * speeling mistake
…i/coderscamp_2021_spa_project_2 into feature/7/application-layout
Feature/7/application layout
* install json-server * create categories.json * fetch data from db.json * add icon picker * add more icons * style icon picker * install axios * add crud functionality for cataegories * change naming convention * add modal * add modal * fix modal use * add validation to category form * general refactor after PR * fix color picker bag and code refactor * remove unnecessary code * add prop types * fix prop types * add env file
* [9][front]add transaction form component * [9][front] add header to Transaction form * [9][front] add useInputState hook and add DatePicker component * [9][front] add TimePicker component * [09][front] add TextInput component and change styling * [9][front] add RadioInput component * [9][front] add NumberInput component * [9][front] fix styles * [9][front] add uplifting inputs state to form * [9][front] change event handling in text and number input * [9][front] add basic styling to buttons * [9] fix functionality, moved state management outside input components * [9] change name of button component * [9] add basic styling * [9] add button styling * [9] delete not needed part * [9] add changes after CR * [9][front] add form validation * [9][front]delete useRef, fix validation * [9][front] fix parenthesis * [9] wrapped TransactionsForm in Modal and add TransactionsPage component * [9] change text on the button * [9] delete validate function from handleSubmit, refactor code * add propTypes * [9] fix propTypes, moved currentDate to separate file * [9] change hardcoded values to props * [9] add yup validation, attempt to display errors * [9] attempt to display error messages * add yup validation, issue with category input and invalid date custom message * [9] resolve app.js conslifct * [9] add validation for categories input * [9] fix validation message for date
* add concurrently to project * . Co-authored-by: Jakub Zakrzewski <[email protected]> Co-authored-by: Jakub Zakrzewski <[email protected]>
* add loader * remove loader from main page
* initial commit * update * update * update * update * update * update * update PropTypes * update function in transaction overview component * update transaction overview component * update * update function Co-authored-by: sylcym <[email protected]> Co-authored-by: sylcym <[email protected]>
* [40] add DropdownItem component, render categories from db to transaction form dropdown * [40]add hook useFetchData * [40] add categories loaded from db to validation schema for transaction form * [40] delete comments
* Added fetch for transactions, added trasactions list component * Removed transactions list component * Removed redundant import * Removed TransactionList in App component * Added pagination, added drop down list to change the number of transactions, addet example list * Updated service for transaction list, removed example list * Removed redundant import * Updated the name of variables and functions * Changed PageNumberButton name to PaginationButton, removed PaginationPageButton component, updated Pagination component, added PaginateButtons component, updated getTransactionsList, updated NumberOfTransactionsWidget * Removed redundant component * Removed redundant component * Updated transicionListServive.js, added .env file
* feature/43/add-reusable-button-component * fix display bug * fix reusable button
* [27] clear main view and moved routes to separate file * [27] rearange navigation component * [27] add icons to navlinks * [27] add more styles to nav * [27] add transactionslist to page transaction component
* Attach components to the transaction list * Added prop types for pagination
* initial commit * update * update * update * update * update * update * update PropTypes * update function in transaction overview component * update transaction overview component * update * update function * fix: change modale action buttons position * update fix: change modale action buttons position * fixed package lock Co-authored-by: Mateusz Ossolinski <[email protected]>
* feature/54/add-globalStyle-and-Theme * add more global styles
* test income/expnse chart * added tranasction from Krzys * proof of concept * pr comments - naing fixes, helper/utils * monthly chart with real data * helpers updates * refactor * dat sync issues fixed
* feature/58/add-categories-to-navigation * add db.json * fix bug in AddCategoryForm * try to fix bug with handleCancel * fix bug wtih handleCancel * change Loader place * fix typo
* Added Typography component * Update Typography Component * Added new prop for Typography Component
* add addNewTransaction function to transactionsListService * [39] fix validation rules * add adding new t5ransaction to db, fix validation * [39] moved imports * [39] make use of transaction service from someone else's file, fix bug * [39] change width of dropdown to fit text * [39] add async await to handle sybmit * [39] moved fetching categories to transactionform * [39] fix issue with rerendering form with change of inputs
* [26] add HomePage component, changed home route in AppRoutes * [26] add basic file structure * [26] add table with latest transactions * [26] add converting given array to 30 last transactions * [26] add scrolling feature to with sticky header to table * add header * add handling empty array, add propTypes * [26] removed mock list * [26] add sorting to latest transactions * [26] changed ternary operator condition * [26] remove scroll in main layout
* feature/66/Transaction-page-styling * fix bug in TransactionPage * fix typos * Change icon color in form
* [26] add HomePage component, changed home route in AppRoutes * [26] add basic file structure * [26] add table with latest transactions * [26] add converting given array to 30 last transactions * [26] add scrolling feature to with sticky header to table * add header * add handling empty array, add propTypes * [67] add context for transactionsList, add global state for transactionsList to App, tested if function setHasDBChanged triggers useEffect in App component * [67] add helper function to handleSubmit * [67] change solution for listTransactions state * [67] fix conflict * [67] fix bug, changed ternary operator back to length as without it does not display error message correctly * [67] fix bug, reverse sorting * [67] changes from CR, create convertDate helper function * [67] add context with transactionsList to TransactionsList component with pagination * [67] add sorting function to transactionsList * create sortByDate function in helpers * [67] move imports to the top
* feature/71/Adjust-modal-styles * fix overlay in mainlayout
* driving chart * one chart per Home & Stats * removed dbjson data * fixed transactionspage import Co-authored-by: Mateusz Ossolinski <[email protected]>
* Add Transactions Overview to Home Page, Updated styles * Updated style for Transactions Overview
* [77] remove height on body and html to remove scroll on page * [77] add overview to homepage and fix issues with table * [77] add piechart to homepage * [77] modified styles for latesttransactions * [77] replace function in pie chart * [77] add reusable text as header in transactions * [77] add convert date to list transactions * [77] add header to categories * [77] separated income from expense monthly transactions chart * [77] attempt to fix heading in t5ransactions * [77] smaller header in t5ransactions * [77] attempt to fix styles on vercel * [77] add context to monthlytransactionchart * [77] attempt to fix monthly chart on vercel
* start * data in the modal; form edit needed * load data, change data working * start on updateTransaction * update * fix edit transaction and categoryPage style * fix create transaction * add action buttons header Co-authored-by: Jakub Zakrzewski <[email protected]>
* stats layout, chart added * proof of concept * v1 * v1 * v1.1 styles * current balance issue * 1.1.1 styles
* [82] add first version of readme * [82] fix typo * [82] fix typo * [82] last fixes * [82] add more transactions to db * [82] change name of the app
develop to master merge 10/02/2022
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/m-ossolinski/coderscamp-2021-spa-project-2/sDzcScxsPo3DdoRjF36gkccYhVUP |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.