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

Improve import functionality #514

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Conversation

Danble
Copy link
Collaborator

@Danble Danble commented Nov 28, 2024

Relevant Issue

(prepend "closes" if issue will be closed by PR)

Summarize what changed in this PR (for developers)

How can the changes be tested?

Please also provide applicable links using relative paths from root (e.g. /apatani/entries) and reviewers can just add that onto preview urls or localhost.

Checklist before marking ready to merge

Please keep it in draft mode until these are completed:

  • Equal time was spent cleaning the code as writing it (Boy Scout Rule)
    • Functions
      • Functions that don't belong in Svelte components are extracted out into .ts files
      • Functions are short and well named
      • Concise tests are written for all functions
    • Classes (a Svelte Component is a Class)
      • Svelte components are broken down into smaller components so that each component is responsible for one thing (Single Responsibility Principle)
      • Stories/variants are written to describe use cases
    • Comments are only included when absolutely necessary information that cannot be explained in code is needed

Copy link

vercel bot commented Nov 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
living-dictionaries ✅ Ready (Inspect) Visit Preview Nov 29, 2024 8:03pm

Copy link

⚡️Lighthouse report

Results for https://living-dictionaries-l79yzqgca-polylingual-development.vercel.app/ (see detailed report)

Category Score
🔴 Performance 16
🟢 Accessibility 90
🟢 Best practices 100
🟢 SEO 96
🟢 PWA 90

Results for https://living-dictionaries-l79yzqgca-polylingual-development.vercel.app/achi/entries (see detailed report)

Category Score
🔴 Performance 45
🟢 Accessibility 90
🟢 Best practices 92
🟢 SEO 97
🟢 PWA 90

Results for https://living-dictionaries-l79yzqgca-polylingual-development.vercel.app/jaRhn6MAZim4Blvr1iEv/entry/yt9ja7ymh9xgba5i (see detailed report)

Category Score
🔴 Performance 40
🟠 Accessibility 89
🟢 Best practices 92
🟢 SEO 100
🟢 PWA 90

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.

1 participant