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

PSP-6895 Digital Documents - Default number of entries on the List View should be 10 instead of 1 #3508

Merged
merged 3 commits into from
Oct 11, 2023

Conversation

asanchezr
Copy link
Collaborator

No description provided.

@asanchezr asanchezr added the bug Something isn't working label Oct 10, 2023
@asanchezr asanchezr self-assigned this Oct 10, 2023
@github-actions
Copy link
Contributor

✅ No secrets were detected in the code.

@codecov
Copy link

codecov bot commented Oct 10, 2023

Codecov Report

Merging #3508 (e6710f8) into dev (193e5e7) will increase coverage by 4.85%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #3508      +/-   ##
==========================================
+ Coverage   70.44%   75.29%   +4.85%     
==========================================
  Files        1341      877     -464     
  Lines       32369    17803   -14566     
  Branches     6019     5000    -1019     
==========================================
- Hits        22802    13405    -9397     
+ Misses       9320     4398    -4922     
+ Partials      247        0     -247     
Flag Coverage Δ
unittests 75.29% <100.00%> (+4.85%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
source/frontend/src/components/Table/Table.tsx 80.00% <100.00%> (ø)
...documents/list/DocumentResults/DocumentResults.tsx 100.00% <ø> (ø)

... and 464 files with indirect coverage changes

@asanchezr asanchezr merged commit 2f0522e into bcgov:dev Oct 11, 2023
9 checks passed
@asanchezr asanchezr deleted the psp-6895-bugfix branch October 13, 2023 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants