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

feat: interactive globe #421

Merged
merged 58 commits into from
Jan 2, 2025
Merged

feat: interactive globe #421

merged 58 commits into from
Jan 2, 2025

Conversation

freds-dev
Copy link
Collaborator

No description provided.

Copy link

socket-security bot commented Oct 27, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@mantine/[email protected] Transitive: environment +5 12.3 MB rtivital
npm/@mantine/[email protected] environment, network 0 1.01 MB rtivital
npm/@playwright/[email protected] None 0 25.4 kB yurys
npm/@radix-ui/[email protected] None +12 298 kB chancestrickland
npm/@radix-ui/[email protected] None +4 155 kB chancestrickland
npm/@radix-ui/[email protected] None +1 32.3 kB chancestrickland
npm/@radix-ui/[email protected] None +4 90.8 kB chancestrickland
npm/@radix-ui/[email protected] None +10 187 kB chancestrickland
npm/@radix-ui/[email protected] None +15 417 kB chancestrickland
npm/@radix-ui/[email protected] None +28 1.38 MB andy-hook, benoitgrelard, chancestrickland, ...3 more
npm/@radix-ui/[email protected] None +20 905 kB chancestrickland
npm/@radix-ui/[email protected] None +1 30.3 kB chancestrickland
npm/@radix-ui/[email protected] None +15 481 kB chancestrickland
npm/@radix-ui/[email protected] None +24 1.03 MB andy-hook, benoitgrelard, chancestrickland, ...3 more
npm/@radix-ui/[email protected] None +14 318 kB chancestrickland
npm/@radix-ui/[email protected] None +9 321 kB chancestrickland
npm/@radix-ui/[email protected] None +28 1.27 MB chancestrickland
npm/@radix-ui/[email protected] None +1 34 kB chancestrickland
npm/@radix-ui/[email protected] None +1 36.9 kB chancestrickland
npm/@radix-ui/[email protected] None +9 141 kB chancestrickland
npm/@radix-ui/[email protected] None +12 281 kB chancestrickland
npm/@radix-ui/[email protected] None +13 426 kB andy-hook, benoitgrelard, chancestrickland, ...3 more
npm/@radix-ui/[email protected] None +11 246 kB chancestrickland
npm/@radix-ui/[email protected] None +4 57.4 kB chancestrickland
npm/@radix-ui/[email protected] None +22 978 kB andy-hook, benoitgrelard, chancestrickland, ...3 more
npm/@remix-run/[email protected] Transitive: environment, filesystem, network, shell, unsafe +205 237 MB
npm/@remix-run/[email protected] None 0 0 B
npm/@remix-run/[email protected] None 0 4.81 kB mjackson
npm/@remix-run/[email protected] Transitive: filesystem, network +10 708 kB
npm/@remix-run/[email protected] Transitive: network +1 2.72 MB
npm/@remix-run/[email protected] None 0 10.2 kB mjackson
npm/@remix-run/[email protected] None 0 3.83 kB mjackson
npm/@remix-run/[email protected] Transitive: network +3 2.79 MB
npm/@tanstack/[email protected] Transitive: environment +1 4.05 MB tannerlinsley
npm/@testing-library/[email protected] environment +1 578 kB testing-library-bot
npm/@types/[email protected] None 0 22.1 kB types
npm/@types/[email protected] None 0 5.02 kB types
npm/@types/[email protected] None 0 8.21 kB types
npm/@types/[email protected] None 0 2.03 MB types
npm/@types/[email protected] None 0 37.9 kB types
npm/@types/[email protected] None +2 1.69 MB types
npm/@vitejs/[email protected] Transitive: environment, filesystem, unsafe +35 11 MB vitebot
npm/@vitest/[email protected] Transitive: environment +8 947 kB vitestbot
npm/[email protected] None +1 5.02 MB chartjs-ci
npm/[email protected] environment, filesystem 0 75.8 kB motdotla
npm/[email protected] environment Transitive: filesystem, shell +8 460 kB ehmicky
npm/[email protected] environment, filesystem, network Transitive: eval +19 799 kB jonchurch
npm/[email protected] None +3 302 kB veltman
npm/[email protected] Transitive: environment +2 28.3 kB

🚮 Removed packages: npm/@mantine/[email protected], npm/@mantine/[email protected], npm/@playwright/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@radix-ui/[email protected], npm/@remix-run/[email protected], npm/@remix-run/[email protected], npm/@remix-run/[email protected], npm/@remix-run/[email protected], npm/@remix-run/[email protected], npm/@remix-run/[email protected], npm/@remix-run/[email protected], npm/@remix-run/[email protected], npm/@tanstack/[email protected], npm/@testing-library/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@types/[email protected], npm/@vitejs/[email protected], npm/@vitest/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected]

View full report↗︎

@freds-dev freds-dev requested a review from Copilot December 11, 2024 15:30

Choose a reason for hiding this comment

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

Copilot reviewed 46 out of 60 changed files in this pull request and generated no suggestions.

Files not reviewed (14)
  • app/components/landing/pricing-plans.tsx: Evaluated as low risk
  • app/components/landing/donate.tsx: Evaluated as low risk
  • app/components/landing/language-selector.tsx: Evaluated as low risk
  • app/components/landing/ger-flag.tsx: Evaluated as low risk
  • app/components/landing/donate-iframe.tsx: Evaluated as low risk
  • app/components/landing/donate-text.tsx: Evaluated as low risk
  • app/components/landing/sections/pricing-plans.tsx: Evaluated as low risk
  • app/components/device-detail/graph.tsx: Evaluated as low risk
  • app/components/device-detail/entry-logs.tsx: Evaluated as low risk
  • app/components/landing/globe.client.tsx: Evaluated as low risk
  • app/components/aggregation-filter.tsx: Evaluated as low risk
  • app/components/landing/sections/features-carousel.tsx: Evaluated as low risk
  • app/components/landing/sections/integrations.tsx: Evaluated as low risk
  • app/components/landing/sections/partners.tsx: Evaluated as low risk
Comments skipped due to low confidence (2)

app/components/device-detail/device-detail-box.tsx:326

  • The data.device.image might be null, which will cause the img tag to fail. Consider providing a default image URL if data.device.image is null.
{data.device.image ? (

app/components/device-detail/device-detail-box.tsx:363

  • The InfoItem component only renders if text is truthy. This could lead to unexpected behavior if text is an empty string or null. Consider handling these cases explicitly.
{data.device.tags?.length > 0 && (
@freds-dev freds-dev marked this pull request as ready for review January 2, 2025 11:20
@freds-dev freds-dev merged commit 5977e2f into dev Jan 2, 2025
7 checks passed
@freds-dev freds-dev deleted the feat/interactive-globe branch January 2, 2025 11:58
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.

2 participants