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

Post sg UI fixes #501

Merged
merged 2 commits into from
Jun 29, 2023
Merged

Post sg UI fixes #501

merged 2 commits into from
Jun 29, 2023

Conversation

danielreti
Copy link
Contributor

Fixes project grid right margin
Fixes summary stats
Fixes analysis runner grid not rendering

@danielreti danielreti changed the base branch from dev to fix-mismatched-sample-count June 27, 2023 03:51
@codecov-commenter
Copy link

codecov-commenter commented Jun 27, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (47bf43a) 71.64% compared to head (da0145a) 71.64%.

❗ Current head da0145a differs from pull request most recent head 7dd2c1f. Consider uploading reports for the commit 7dd2c1f to get more accurate results

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@                     Coverage Diff                      @@
##           fix-mismatched-sample-count     #501   +/-   ##
============================================================
  Coverage                        71.64%   71.64%           
============================================================
  Files                               90       90           
  Lines                             7686     7686           
============================================================
  Hits                              5507     5507           
  Misses                            2179     2179           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Collaborator

@illusional illusional left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Can you put the link to the tutorial in the css as a reference for padding an overflow container?

@danielreti danielreti merged commit 64b3f6c into fix-mismatched-sample-count Jun 29, 2023
@danielreti danielreti deleted the post-sg-ui-fixes branch June 29, 2023 01:54
danielreti added a commit that referenced this pull request Jun 29, 2023
* fixed API turning all smaples as true and fixed project grid showing all samples not just active ones

* linting

* moved active check

* moved active type conversion to sample model

* Post sg UI fixes (#501)

* small fixes to project and analysis grids

* Added link to css overflow blogpost
vivbak added a commit that referenced this pull request Jul 18, 2023
* Fix Mismatched Samples count (#498)

* fixed API turning all smaples as true and fixed project grid showing all samples not just active ones

* linting

* moved active check

* moved active type conversion to sample model

* Post sg UI fixes (#501)

* small fixes to project and analysis grids

* Added link to css overflow blogpost

* Update backup script to fetch new credentials (#502)

* Fix get analysis by id query (#503)

* Fix get analysis by ID (+ simplify analysis tests)

* Linting

* Cohort builder schema (#497)

* Cohort builder schema

* Fix bad copy-paste

* Fix nullable specification

* Add internal pid to ParticipantUpsert constructor from ParsedParticipant object (#505)

* Add blob.reload() call to cloudhelper get_gcs_blob (#506)

* Audit module and test update for SG changes (#509)

* Audit module and test update for SG changes

* Remove datasets list from audithelper

* Typehint fixes for linting

* Add project and sample id filter (#511)

* Allows uses to filter on sample ID within a project

* remove prints

* Bump version: 6.0.6 -> 6.1.0 (#512)

* Update deploy.yaml (#517)

* Update deploy.yaml

* Ignore pyenv and dotenv files

* update names to kebab-case

* add new line to make end-of-file-fixer happy

* ignore mypy cache

---------

Co-authored-by: daniaki <7043686+daniaki@users.noreply.github.com>

---------

Co-authored-by: danielreti <47542969+danielreti@users.noreply.github.com>
Co-authored-by: Michael Franklin <michael.franklin@populationgenomics.org.au>
Co-authored-by: EddieLF <34049565+EddieLF@users.noreply.github.com>
Co-authored-by: daniaki <7043686+daniaki@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants