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

Remove unknown=EXCLUDE after devnet fix response #1323

Closed
tkumor3 opened this issue Mar 14, 2024 · 1 comment · Fixed by #1391
Closed

Remove unknown=EXCLUDE after devnet fix response #1323

tkumor3 opened this issue Mar 14, 2024 · 1 comment · Fixed by #1391
Assignees
@kkawula
Labels
priority: low

Comments

@tkumor3
Copy link
Contributor

tkumor3 commented Mar 14, 2024 •
edited
Loading

For getStateUpdate andgetBlockWithTxHashes devnet returns a response that contains more fields than should. Remove unknown=EXCLUDE when Devnet will fix this issue. Also set L1HandlerTransactionTraceSchema execution_resources as required after starknet update starknet-spec
@github-actions GitHub Actions
Copy link
Contributor

This issue is stale because it has not received any activity in the last 30 days. Remove stale label or add a comment, otherwise it will be closed in 5 days.

@github-actions github-actions bot added the stale label Apr 18, 2024
@ddoktorski ddoktorski removed the stale label Apr 24, 2024
@ddoktorski ddoktorski mentioned this issue May 10, 2024
Merged
1 task
@ddoktorski ddoktorski removed the blocked label Jul 9, 2024
@kkawula kkawula mentioned this issue Jul 10, 2024
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kkawula kkawula

Labels
priority: low
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove unknown=EXCLUDE after devnet fix response software-mansion/starknet.py
4 participants
@THenry14 @tkumor3 @ddoktorski @kkawula