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

fix: Fix crash on initial slot fill without a former question asked #1253

Merged
merged 1 commit into from
Jan 12, 2023

Conversation

Apollon77
Copy link
Contributor

Pull Request Template

PR Checklist

  • I have run npm test locally and all tests are passing.
  • I have added/updated tests for any new behavior.
  • If this is a significant change, an issue has already been created where the problem / solution was discussed: [N/A, or add link to issue here]

PR Description

fixes #1252

@Apollon77
Copy link
Contributor Author

@eric-lara should be released as fix ideally soon, because was introduce by my changes. sorry

@Apollon77
Copy link
Contributor Author

@eric-lara I verified while writing the tests that they all fail with the error from the issue befroe the fx and work with the fix

@sonarcloud
Copy link

sonarcloud bot commented Jan 8, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell B 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@Apollon77
Copy link
Contributor Author

And it seems the testing want to be restarted again for nodejs 16

@ericzon ericzon merged commit e968bef into axa-group:master Jan 12, 2023
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.

Cannot read properties of undefined (reading 'latestSlot')
2 participants