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

Error deleting instance with missing subscription #2080

Closed
knolleary opened this issue May 3, 2023 · 0 comments · Fixed by #2081
Closed

Error deleting instance with missing subscription #2080

knolleary opened this issue May 3, 2023 · 0 comments · Fixed by #2081
Assignees
@knolleary
Labels
bug Something isn't working size:XS - 1 Sizing estimation point
Milestone
1.7

Comments

@knolleary
Copy link
Member 

TypeError: Cannot read properties of undefined (reading 'log')
    at Object.removeProject (/Users/nol/code/ff/flowforge-dev-env/packages/flowforge/forge/ee/lib/billing/index.js:291:42)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async SubscriptionHandler.removeProject (/Users/nol/code/ff/flowforge-dev-env/packages/flowforge/forge/containers/wrapper.js:84:13)
    at async Object.remove (/Users/nol/code/ff/flowforge-dev-env/packages/flowforge/forge/containers/wrapper.js:209:17)
    at async Object.<anonymous> (/Users/nol/code/ff/flowforge-dev-env/packages/flowforge/forge/routes/api/project.js:319:13)

Caused by incorrect reference to app here:

https://github.com/flowforge/flowforge/blob/cf70d3b84ae755effb5e5bd285364dbc33008295/forge/ee/lib/billing/index.js#L291
@knolleary knolleary added the size:XS - 1 Sizing estimation point label May 3, 2023
@knolleary knolleary self-assigned this May 3, 2023
@knolleary knolleary added the bug Something isn't working label May 3, 2023
knolleary added a commit that referenced this issue May 3, 2023
@knolleary
Fix deleting instance with missing subscription
eaa729e 
Fixes #2080
@knolleary knolleary mentioned this issue May 3, 2023
Merged
@MarianRaphael MarianRaphael added this to the 1.7 milestone May 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@knolleary knolleary

Labels
bug Something isn't working size:XS - 1 Sizing estimation point
Projects
🛠 Development
Archived in project
Milestone
1.7
Development

Successfully merging a pull request may close this issue.

Fix deleting instance with missing subscription FlowFuse/flowfuse
2 participants
@knolleary @MarianRaphael