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

Cde create visualization page #340

Merged
merged 40 commits into from
May 16, 2024

Conversation

edlu77
Copy link
Contributor

@edlu77 edlu77 commented Apr 2, 2024

@edlu77 edlu77 requested review from axdanbol and bherr2 April 2, 2024 17:30
@edlu77 edlu77 self-assigned this Apr 2, 2024
Copy link

nx-cloud bot commented Apr 2, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 06847ff. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


🟥 Failed Commands
nx affected --target=test --parallel=3 --exclude=tag:karma-test-runner --ci --code-coverage
✅ Successfully ran 5 targets

Sent with 💌 from NxCloud.

@axdanbol axdanbol changed the base branch from develop to cell-distance-explorer April 2, 2024 18:19
Copy link
Contributor

@axdanbol axdanbol left a comment

Choose a reason for hiding this comment

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

Best practices and some changes

@axdanbol
Copy link
Contributor

axdanbol commented Apr 2, 2024

Please pull and resolve conflicts from cell-distance-explorer. Doing so will allow ci and preview scripts to run.

Copy link

github-actions bot commented Apr 2, 2024

🚀 Preview Deploy Report Updated

✅ Successfully deployed preview here

</mat-card-title>
</mat-card-header>
<mat-card-actions>
<div class="row-1">
Copy link
Contributor

Choose a reason for hiding this comment

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

Use a hidden element after age input on larger breakpoints

</mat-card-title>
</mat-card-header>
<mat-card-actions>
<div class="row-1">
Copy link
Contributor

Choose a reason for hiding this comment

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

Use flex with flex-wrap

@axdanbol axdanbol marked this pull request as ready for review April 17, 2024 19:42
Copy link

sonarcloud bot commented May 16, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@axdanbol axdanbol merged commit c89d973 into cell-distance-explorer May 16, 2024
8 of 9 checks passed
@axdanbol axdanbol deleted the cde-create-visualization-page branch May 16, 2024 21:49
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