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(store): add typing to default vals #730

Merged
merged 5 commits into from
Oct 18, 2024
Merged

fix(store): add typing to default vals #730

merged 5 commits into from
Oct 18, 2024

Conversation

Flemmli97
Copy link
Contributor

What this PR does 📖

  • Adds typing to default values for sanity check. Removing some unneeded values

@github-actions github-actions bot added the Missing dev review Two dev reviews are required on PR label Oct 17, 2024
Copy link

github-actions bot commented Oct 17, 2024

Download the app installers for this pull request:

@github-actions github-actions bot added the Failed Automated Test Failed Automated Tests CI label Oct 17, 2024
Copy link

github-actions bot commented Oct 17, 2024

Automated tests execution is complete! You can find the Playwright test report here and the Allure Test Report here

@github-actions github-actions bot removed the Failed Automated Test Failed Automated Tests CI label Oct 18, 2024
@phillsatellite phillsatellite added QA Approved PR has been tested by QA Team and removed Missing dev review Two dev reviews are required on PR labels Oct 18, 2024
@phillsatellite phillsatellite merged commit 2da5b43 into dev Oct 18, 2024
7 checks passed
@phillsatellite phillsatellite deleted the default-type branch October 18, 2024 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QA Approved PR has been tested by QA Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants