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

Sync releases #531

Merged
merged 39 commits into from
Nov 13, 2024
Merged

Sync releases #531

merged 39 commits into from
Nov 13, 2024

Conversation

amattu2
Copy link
Member

@amattu2 amattu2 commented Nov 13, 2024 •
edited
Loading

Overview

Git handled the merge automatically. The only manual migration is 7282f07.

Alejandro-Vega and others added 30 commits October 31, 2024 10:13
@Alejandro-Vega
Updated listSubmissions query and response type
fa90a41
@Alejandro-Vega
Update filters to use org from listSubmissions API instead of retriev…
b9b5c71 
…ing all orgs
@Alejandro-Vega
Update tests for filters
4a61edf
@Alejandro-Vega
Add additional tests
9eae029
@amattu2
Synchronize CDE parsing logic
aacfdad
@Alejandro-Vega
Merge branch '3.1.0' into CRDCDH-1738-org-dropdown
1aa6002
@Alejandro-Vega
Sort orgs by name
607acf4
@Alejandro-Vega
Remove any mention of orgs from Data Submission List page
5698b86
@Alejandro-Vega
Fix collaborators property height
26265ad
@Alejandro-Vega
Fixed resetting table page and filters race condition when page is no…
8b6a6d9 
…t the first page
@Alejandro-Vega
Remove useEffect and replace with linear logic
2d9e417
@Alejandro-Vega
Remove unused import
cbbfa1d
@Alejandro-Vega
Update additional state on refetch after creating a new submission
ad31897
@amattu2
feat: Use CDE Origin from MDF
1a405f6
@amattu2
fix: Remove CDEOrigin from input
9d3bf8f
@Alejandro-Vega
Undo previous change, didn't fix bug
2eb7c13
@Alejandro-Vega
Remove unused imports
d32e03b
@Alejandro-Vega
Bandage fix for debouncing specific dropdowns
efdb01f
@Alejandro-Vega
Add waitFor to tests
b12f2ba
@Alejandro-Vega
Merge pull request #521 from CBIIT/NO-ISSUE/CDE-Parsing-Logic-Sync
43e9148 
Synchronize CDE parsing logic
@Alejandro-Vega
Merge branch '3.1.0' into CRDCDH-1738-org-dropdown
0323702
@Alejandro-Vega
Update org type
99a458b
@Alejandro-Vega
Merge pull request #516 from CBIIT/CRDCDH-1738-org-dropdown
5378f55 
CRDCDH-1738 Data Submission List - Unlock Organization dropdown
@amattu2
Restyle metric field per UX
21be73e
@amattu2
fix: Node labels overlapping
3aae75a
@amattu2
CRDCDH-1977 Fix Model Navigator crash and add test coverage
d6f6d15
@amattu2
fix: Focused input missing border radius
b8cd159
@Alejandro-Vega
Merge pull request #526 from CBIIT/CRDCDH-1976
aa06d99 
CRDCDH-1976 Fix Node labels overlapping
@Alejandro-Vega
Merge branch '3.1.0' into CRDCDH-1428
c706dcd
@Alejandro-Vega
Merge pull request #525 from CBIIT/CRDCDH-1428
573ba6a 
Restyle metric field per UX
@coveralls
Copy link
Collaborator

coveralls commented Nov 13, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 11823554510

Details

  • 43 of 60 (71.67%) changed or added relevant lines in 8 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.3%) to 53.812%
Changes Missing Coverage Covered Lines Changed/Added Lines %
src/hooks/useBuildReduxStore.ts 0 1 0.0%
src/content/dataSubmissions/Controller.tsx 0 2 0.0%
src/components/DataSubmissions/DataSubmissionListFilters.tsx 15 19 78.95%
src/content/dataSubmissions/DataSubmissionsListView.tsx 0 10 0.0%
Files with Coverage Reduction New Missed Lines %
src/content/dataSubmissions/DataSubmissionsListView.tsx 1 0.0%
Totals Coverage Status
Change from base Build 11748408531: 0.3%
Covered Lines: 3689
Relevant Lines: 6360
💛 - Coveralls

@amattu2 amattu2 merged commit 106fc23 into 3.2.0 Nov 13, 2024
5 checks passed
@amattu2 amattu2 deleted the sync-releases branch November 13, 2024 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@amattu2 @coveralls @Alejandro-Vega