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

types: add missing loadingScreen type definition #8648

Merged
merged 2 commits into from
Jan 14, 2021
Merged

types: add missing loadingScreen type definition #8648

merged 2 commits into from
Jan 14, 2021

Conversation

gekkedev
Copy link

@gekkedev gekkedev commented Jan 14, 2021

Types of changes

  • Bug fix (a non-breaking change which fixes an issue)
  • New feature (a non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Description

This PR adds the missing type definition for loadingScreen as mentioned in #6272. The feature was implemented but that made type definition incompatible (if you decide to overwrite this setting).

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly. (PR: #)
  • I have added tests to cover my changes (if not applicable, please state why)
  • All new and existing tests are passing.

@pi0 pi0 changed the title added loadingScreen typedef fix(types): add missing loadingScreen type definition Jan 14, 2021
@pi0 pi0 changed the title fix(types): add missing loadingScreen type definition types: add missing loadingScreen type definition Jan 14, 2021
Co-authored-by: pooya parsa <pyapar@gmail.com>
@pi0 pi0 merged commit 78ba994 into nuxt:dev Jan 14, 2021
@pi0
Copy link
Member

pi0 commented Jan 14, 2021

Thanks!

@gekkedev
Copy link
Author

Welcome; glad I could contribute something!

@gekkedev gekkedev deleted the patch-1 branch January 14, 2021 18:40
@pi0 pi0 mentioned this pull request Feb 9, 2021
@danielroe danielroe added the 2.x label Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants