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.

Sign up for GitHub

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

Jump to bottom

chris update equipment detail #827

Closed
wants to merge 90 commits into from
Closed

chris update equipment detail #827

wants to merge 90 commits into from

Conversation

chrischenlixing
Copy link
Contributor

Related PRS:

To test this backend PR you need to check out the #XXX frontend PR.
…

Main changes explained:

  • Created routes, APIs and database models for the equipment details page

How to test:

  1. check into the current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log in as an admin user
  5. go to dashboard→ Tasks→ task→…
  6. verify function “A” (feel free to include a screenshot here)

navneeeth and others added 30 commits December 26, 2023 12:20
@navneeeth
remove sangam's email from the recipient list
e991883
@navneeeth
add google doc link
92c0c44
@navneeeth
added google doc link to the email body
7e31b26
@navneeeth
spacing fix
22639ac
@navneeeth
Merge branch 'development' of https://github.com/OneCommunityGlobal/H…
e5d6a9f 
…GNRest into navneeth_add_google_doc_link_to_weekly_summaries_email
@navneeeth
update adminLinks check for google drive link
dc55c93
@navneeeth
Merge branch 'development' of https://github.com/OneCommunityGlobal/H…
a61c144 
…GNRest into navneeth_add_google_doc_link_to_weekly_summaries_email
@navneeeth
update adminLinks check and revert to single quotes in emailSender pa…
e5684a3 
…rameters
@navneeeth
update googleDocLink retrieval logic
4a88a56
@ChengyanOo
Merge branch 'development' into Chengyan-Phase2-Reusables-List
b1c4bbd
@ChengyanOo
Merge branch 'development' into Chengyan-Phase2-Reusables-List
53156f6
@navneeeth
merge with development
3600601
@navneeeth
Merge branch 'development' of https://github.com/OneCommunityGlobal/H…
d4b6b17 
…GNRest into navneeth_add_google_doc_link_to_weekly_summaries_email
@navneeeth
set up dev mode logging in the logger implementation
4c0f1d6
@navneeeth
fix build issue
0ec5c6f
@DiegoSalas27
chore: enforcng eslint and prettier on staged files
dbb10bf
@ChengyanOo
Merge branch 'development' into Chengyan-Phase2-Reusables-List
80beeba
@abdel-lall
fix blue square reason email
02afd3d
@one-community
Merge pull request #801 from OneCommunityGlobal/Fix_blue_square_reaso…
200e358 
…n_email

fix blue square reason email
@DiegoSalas27
chore: update prepare command to allow pre-commit hook to be executable
271c7b0
@DiegoSalas27
build: rollback to previous version of eslint-plugin-import
dde92e4
@DiegoSalas27
fix: update package-lock.json
18efc1a
@DiegoSalas27
chore: add conventional commits with husky
0a853b1
@DiegoSalas27
build: update husky dependency
26f92f3
@DiegoSalas27
build: update linting dependencies
9839877
@DiegoSalas27
build: update linting dependencies
4ebf91f
@DiegoSalas27
merged dependency changes
b76b1e1
@DiegoSalas27
chore: add jest
f3a1e39
@DiegoSalas27
build: updated dependencies
167aae3
@one-community
Merge pull request #803 from OneCommunityGlobal/diego_add_eslint_and_…
7292e3d 
…prettier

Diego enforcing eslint and prettier on staged files
DiegoSalas27 and others added 28 commits March 26, 2024 21:34
@DiegoSalas27
test: add integration test to postUserProfile
643a8d2
@DiegoSalas27
test: Ensure getUserProfiles returns 400 if the user doesn't have get…
791d2f1 
…UserProfiles permission
@Abi-Liu
refactor: refactored server.js to be make integration testing easier …
02f2dc9 
…via supertest
@Abi-Liu
test: Integration tests for postactionItem route
9dd8052
@Abi-Liu
chore: removed unnecessary comments
1fcfbe6
@Abi-Liu
test: integration tests for getActionItem routes
8f12e08
@DiegoSalas27
test: Ensure getUserProfiles returns 500 if there are no users in the…
d948ef5 
… database and the allusers key doesn't exist in NodeCache
@Abi-Liu
refactor: refactored deleteactionItem method
d3fd05e
@Abi-Liu
test: deleteactionItem unit tests
b6c65fe
@DiegoSalas27
test: Ensure getUserProfiles returns 404 if any error occurs while ge…
31e33f8 
…tting all user profiles
@Abi-Liu
refactor: refactored editactionItem method
01bea8b
@Abi-Liu
test: unit tests for the editactionItem method inside the actionItemC…
843c91c 
…ontroller
@DiegoSalas27
test: Ensure getUserProfiles returns 200 if there are no users in the…
b589c86 
… database and the allusers key exists in NodeCache
@Abi-Liu
chore: removed .only from actionItemController unit tests
9fe7990
@DiegoSalas27
test: Ensure getUserProfiles returns 200 if there are users in the da…
739c5c3 
…tabase
@one-community
Merge pull request #813 from OneCommunityGlobal/diego_add_unit_test_p…
58d8e78 
…ostUserProfile

Diego finished adding tests to postUserProfile
@Abi-Liu
test: delete action item route integration tests
5dc6295
@Abi-Liu
test: putActionItem integration tests
ef339ec
@Abi-Liu
test: ensure notificationcreated method is called with the proper arg…
ae6198a 
…uments
@Abi-Liu
test: testing arguments that dependencies are called with to ensure c…
7737e2e 
…orrectness
@Abi-Liu
test: assert that all dependencies in editactionItem method are calle…
8d8daf5 
…d with correct arguments
@DiegoSalas27
refactor: update tests to cover dependencies being called with correc…
75ec8a2 
…t parameters
@DiegoSalas27
chore: merge development into diego_add_unit_test_getUserProfiles
2e2e121
@DiegoSalas27
chore: merged development into abi_getactionItem_tests
9048f37
@DiegoSalas27
Merge pull request #817 from OneCommunityGlobal/abi_getactionItem_tests
e936017 
Abi Unit and Integration tests for getActionItem routes and Controllers
@DiegoSalas27
test: adding missing integration test for getUserProfiles
e2b998f
@DiegoSalas27
docs: update getUserProfiles-usecase
6c6c015
@DiegoSalas27
Merge pull request #822 from OneCommunityGlobal/diego_add_unit_test_g…
b3599a4 
…etUserProfiles

Diego add unit test get user profiles
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@one-community one-community Awaiting requested review from one-community

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@chrischenlixing @navneeeth @ChengyanOo @DiegoSalas27 @abdel-lall @one-community @Abi-Liu @wang9hu