Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

MVP-1 -> MVP-2 #154

Merged
merged 128 commits into from
Oct 4, 2023
Merged

MVP-1 -> MVP-2 #154

merged 128 commits into from
Oct 4, 2023

Conversation

amattu2
Copy link
Member

@amattu2 amattu2 commented Sep 28, 2023 •
edited
Loading

amattu2 and others added 30 commits August 30, 2023 16:29
@amattu2
CRDCDH-178/CRDHDH-179 Add User Management
1edb76c
@amattu2
Remove unimplemented test suite
4b14b11
@amattu2
Remove unused typedef
6f8e104
@amattu2
Implement BE error messages, add Status filter
925702e
@amattu2
CRDCDH-179 Hide save button for Org Owners
3927024
@amattu2
Skip Organization preloading for non-admin roles
03616de
@Alejandro-Vega
Updated dbGaP label in section B
206dbc4
@Alejandro-Vega
Styling updates, mostly colors and spacing
b64a6e7
@jonkiky
Merge pull request #106 from CBIIT/CRDCDH-179-179-Squashed
88a502c 
CRDCDH-178/CRDHDH-179 Account Management
@amattu2
CRDCDH-179 Fix user table count
5b9dfcf
@jonkiky
Merge pull request #108 from CBIIT/CRDCDH-318
63c62bb 
CRDCDH-318
@jonkiky
Merge pull request #110 from CBIIT/CRDCDH-178-BugFix
c97851f 
CRDCDH-178 Fix user table count
@amattu2
CRDCDH-178 Fix wrong count variable
94cb032
@amattu2
Revert rowPerPageOptions prop
b0bee97
@mtangmt
changing last updated date to n/a if new
15270d6
@amattu2
CRDCDH-178 Set page on filter/sort
760b2bf
@mtangmt
update status bar to always display date
68bf1ee
@mtangmt
remove unneeded data
e3086eb
@mtangmt
reverting NA changes and updating test
25273b8
@mtangmt
remove old eslint override
a5568b3
@amattu2
CRDCDH-217 Design QA changes and fixes
2366f20
@amattu2
CRDCDH-217 Update disabled button
896d3c8
@mtangmt
stashing changes
08911ce
@amattu2
Update Page Titles per Comment 17
8aca77a
@mtangmt
widened and added shadows to table autocompletes
6eedff2
@amattu2
CRDCDH-336 Monitor custom input for onChange event
b5911f4
@Alejandro-Vega
Update DatePickerInput error messages. Also, updated default value su…
9fb65d7 
…bmitted to prevent form detected change
@Alejandro-Vega
Fixed various bugs for DatePickerInput. Allowed empty dates and fixed…
dbc9432 
… error validation
@Alejandro-Vega
Reverted some of the DatePickerInput changes. Added additional onChaq…
f7573ee 
…nge to the textfield directly. Allowed for validation to pass when input is the same as format pattern or empty
@mtangmt
change hover color to 3E7E6D
19ed24b
Alejandro-Vega and others added 16 commits September 26, 2023 10:54
@Alejandro-Vega
Updated GenericAlert to allow different background colors for differe…
cb736d0 
…nt severity levels with a default as success
@Alejandro-Vega
Updated Saving to catch errors and check if it is due to duplicate st…
91771e7 
…udy abbrev. then show different alert error
@Alejandro-Vega
Fixed ProgressBar flickering during saving and an error occurs
962a4c4
@mtangmt
Adding error msg on login fail
98f2192
@mtangmt
updating footer email error to match CRDC
5a34b41
@Alejandro-Vega
Updated funding agency id's to include index
da3af13
@Alejandro-Vega
Added id's to elements containing content in Review page
ec63fba
@jonkiky
Merge pull request #142 from CBIIT/CRDCDH-398
a446b5f 
CRDCDH-398 Unique Study Abbreviation
@jonkiky
Merge pull request #143 from CBIIT/CRDCDH-433
8a81a92 
Adding error msg on login fail
@jonkiky
Merge pull request #144 from CBIIT/CRDC-434
97af67d 
CRDCDH-434 match footer email validation to CRDC
@jonkiky
Merge pull request #145 from CBIIT/CRDCDH-437
3a9cc86 
CRDCDH-437 Adding Id's to Review page
@mtangmt
fixing login issues
8946b3d
@jonkiky
Merge pull request #150 from CBIIT/CRDCDH-433-bugfix
856c793 
fixing login issues
@amattu2
Fix CVE-2023-4863 and CVE-2023-38039
63c949d
@jonkiky
Merge pull request #151 from CBIIT/amattu2/CRDCDH-421-427
6350204 
Fix CVE-2023-4863 and CVE-2023-38039
@amattu2
UX QA additional changes
2ebcfdc 
Comments:
- 37 (Adjusted)
- 44 (More changes)
- 50 (New)
amattu2 and others added 9 commits September 28, 2023 11:35
@amattu2
Migrate to Questionnaire styling
4a7ad13
@Alejandro-Vega
Allow for additional saves to occur when error due to duplicate study…
1cd73ac 
… abbrev even if no changes were made
@jonkiky
Merge pull request #153 from CBIIT/amattu2/CRDCDH-351-V2
d94dbd9 
CRDCDH-351 UX QA additional changes
@mtangmt
update header logo to closer match CRDC
8671fbd
@jonkiky
Merge pull request #157 from CBIIT/CRDCDH-399-Header-Logo-update
3441054 
Crdcdh 399 header logo update
@Alejandro-Vega
Updated form so that user has to fix Study Abbreviation error before …
11088da 
…proceeding to other sections
@Alejandro-Vega
Fixed bug where "Next" button on section D wouldn't undisable when al…
4560f18 
…l sections are complete in New form
@Alejandro-Vega
Remove unnecessary state, and add back saving status
2fcbc3b
@jonkiky
Merge pull request #155 from CBIIT/CRDCDH-449
e2ca31b 
CRDCDH-449 Allow additional saves when duplicate Study Abbrev error
@amattu2 amattu2 marked this pull request as ready for review October 4, 2023 15:48
@amattu2 amattu2 merged commit 64905e1 into mvp-2 Oct 4, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@jonkiky jonkiky Awaiting requested review from jonkiky

@mtangmt mtangmt Awaiting requested review from mtangmt

@Alejandro-Vega Alejandro-Vega Awaiting requested review from Alejandro-Vega

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

Successfully merging this pull request may close these issues.
4 participants
@amattu2 @Alejandro-Vega @jonkiky @mtangmt