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.

Sign up for GitHub

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

Jump to bottom

feat(platform): Cancel app data storage service #2797

Merged
merged 22 commits into from
Jan 19, 2024
Merged

feat(platform): Cancel app data storage service #2797

merged 22 commits into from
Jan 19, 2024

Conversation

Cosmin-Parvulescu
Copy link
Contributor

@Cosmin-Parvulescu Cosmin-Parvulescu commented Dec 22, 2023 •
edited
Loading

Description

  • Updated storage screen to show information on current package
  • Updated storage screen to allow cancellation of packages and to reflect the DELETING state
  • Updated wrangler to 3.18 alongside the cftypes; aligned it in entire monorepo
  • Added extra functionality to Spinner
  • Implemented core queue producer / consumer specialsauce pattern (here be dragons)
  • Added ExternalAppDataDelReq and ExternalAppDataDelSignal to the queue (here also be dragons)
  • Updated UsageKV wrangler configuration to actually work for higher envs than local
  • Added queue wrangler configs for all envs
  • Updated DO representation of ExternalAppDataPackage to include both details as well as status (ENABLED / DELETING)
  • Updated setExternalAppDataPackage to use new definitions
  • Added two SQL scripts to help populate authorizations for development / testing help

Related Issues

Testing

Screen.Recording.2024-01-11.at.14.58.27.mov

Checklist

  • I have read the CONTRIBUTING guidelines
  • I have tested my code (manually and/or automated if applicable)
  • I have updated the documentation (if necessary)

@Cosmin-Parvulescu Cosmin-Parvulescu added enhancement Indicates new feature requests good-first-issue Indicates a good issue for first-time contributors labels Dec 22, 2023
@Cosmin-Parvulescu Cosmin-Parvulescu self-assigned this Dec 22, 2023
@Cosmin-Parvulescu Cosmin-Parvulescu force-pushed the feat/platform/Cancel-app-data-storage-service branch 2 times, most recently from 5ef3339 to da6b144 Compare January 11, 2024 12:41
@Cosmin-Parvulescu Cosmin-Parvulescu marked this pull request as ready for review January 11, 2024 13:00
@Cosmin-Parvulescu
feat(platform): Cancel app data storage service
95cd091
@Cosmin-Parvulescu
Add batch queue functionality and logging
ca11ddf
@Cosmin-Parvulescu
Revert "Add batch queue functionality and logging"
1f3c8fa 
This reverts commit bfc27e5403c71256f9c7548f637245ebc5a60bbd.
@Cosmin-Parvulescu
Add storage form and buttons for enabling/disabling app data package
a1d3a1e
@Cosmin-Parvulescu
Refactor storage.ostrich.tsx to handle null values in appDetails.exte…
e32bd03 
…rnalAppDataPackageDefinition
@Cosmin-Parvulescu
Upgrade wrangler
f0ca007
@Cosmin-Parvulescu
Add queue functionality for deleting app data
48eccc9
@Cosmin-Parvulescu
Refactor external app data package definition and cancellation
948a990
@Cosmin-Parvulescu
Add ExternalAppDataPackageStatus enum
2f8f960
@Cosmin-Parvulescu
Refactor authorization and starbase nodes, and update storage.ostrich…
b00fed1 
….tsx
@Cosmin-Parvulescu
Update storage.ostrich.tsx with comments on package usage
27cd626
@Cosmin-Parvulescu
Update limit constant value to 2000
4076f0b
@Cosmin-Parvulescu
Add sync queue configuration to dev environment
8ce8d2a
@Cosmin-Parvulescu
Add script to generate test data for app authorizations
c77233b
@Cosmin-Parvulescu
Add script to generate nodes and edges for testing purposes
a69b6ad
@Cosmin-Parvulescu
Cleanup
5c228ee
@Cosmin-Parvulescu
Add Spinner component for service cancellation progress
4d31a5f
@Cosmin-Parvulescu
Added cancellation modal
984a21c
@Cosmin-Parvulescu
Update core-queue configuration
70800cf
@Cosmin-Parvulescu
Refactor fetch function and remove console logs
2e862ca
@Cosmin-Parvulescu
Update kv_namespaces in wrangler.current.toml
2993c9f
@Cosmin-Parvulescu Cosmin-Parvulescu force-pushed the feat/platform/Cancel-app-data-storage-service branch from 45c2c36 to 2993c9f Compare January 19, 2024 14:57
@betimshahini betimshahini merged commit d7fb632 into main Jan 19, 2024
15 of 16 checks passed
@betimshahini betimshahini deleted the feat/platform/Cancel-app-data-storage-service branch January 19, 2024 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@betimshahini betimshahini betimshahini approved these changes

Assignees

@Cosmin-Parvulescu Cosmin-Parvulescu

Labels
enhancement Indicates new feature requests good-first-issue Indicates a good issue for first-time contributors
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feat(platform): Cancel app data storage service
2 participants
@Cosmin-Parvulescu @betimshahini