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

Github + OPFS delete + OPFS tests #977

Merged
merged 12 commits into from Feb 9, 2024

Conversation

nickcastel50
Copy link
Contributor

This PR adds code (currently not hooked up to UI) for deleting models from OPFS and GitHub. It also adds a utility function to check to see if a file exists in OPFS.

Additionally, tests have been added for all OPFS related functions and the service worker has been mocked.

@nickcastel50 nickcastel50 added the enhancement New feature or request label Feb 8, 2024
@nickcastel50 nickcastel50 self-assigned this Feb 8, 2024
Copy link

netlify bot commented Feb 8, 2024

Deploy Preview for bldrs-share ready!

Name Link
🔨 Latest commit eff686a
🔍 Latest deploy log https://app.netlify.com/sites/bldrs-share/deploys/65c6883f23127a0008ec96c1
😎 Deploy Preview https://deploy-preview-977--bldrs-share.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 25 (🔴 down 1 from production)
Accessibility: 80 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Feb 8, 2024

Deploy Preview for bldrs-share-v2 ready!

Name Link
🔨 Latest commit eff686a
🔍 Latest deploy log https://app.netlify.com/sites/bldrs-share-v2/deploys/65c6883ff1211f00089819df
😎 Deploy Preview https://deploy-preview-977--bldrs-share-v2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

src/OPFS/OPFS.worker.js Outdated Show resolved Hide resolved
src/OPFS/OPFS.worker.js Show resolved Hide resolved
src/OPFS/OPFS.worker.js Show resolved Hide resolved
src/OPFS/OPFSService.js Outdated Show resolved Hide resolved
src/OPFS/utils.js Outdated Show resolved Hide resolved
src/OPFS/utils.test.js Outdated Show resolved Hide resolved
src/OPFS/utils.test.js Show resolved Hide resolved
src/utils/GitHub.js Outdated Show resolved Hide resolved
src/utils/GitHub.js Show resolved Hide resolved
src/utils/GitHub.js Outdated Show resolved Hide resolved
src/OPFS/OPFS.worker.js Show resolved Hide resolved
src/OPFS/OPFS.worker.js Show resolved Hide resolved
src/OPFS/OPFS.worker.js Outdated Show resolved Hide resolved
src/OPFS/OPFSService.js Show resolved Hide resolved
src/OPFS/OPFSService.js Show resolved Hide resolved
src/utils/GitHub.js Show resolved Hide resolved
@nickcastel50 nickcastel50 merged commit 436e8b5 into bldrs-ai:main Feb 9, 2024
13 checks passed
Copy link

sentry-io bot commented Feb 14, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ HttpError: Not Found getLatestCommitHash(src/utils/GitHub) View Issue

Did you find this useful? React with a 👍 or 👎

@OlegMoshkovich OlegMoshkovich added this to the Versions milestone Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants