Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Apr 29, 2025. It is now read-only.

Conversation

@danielroe
Copy link
Owner

@danielroe danielroe commented Feb 13, 2023

The aim is that the time component doesn't immediately update when hydrated, but only if a subsequent change to the time is detected, so Date.now() can be used in a server template and be 'frozen' at the time of render unless subsequently updated.

cc: @userquin

@danielroe danielroe added the enhancement New feature or request label Feb 13, 2023
@danielroe danielroe self-assigned this Feb 13, 2023
@danielroe danielroe marked this pull request as draft February 13, 2023 10:23
@codecov-commenter
Copy link

Codecov Report

Base: 77.60% // Head: 77.04% // Decreases project coverage by -0.56% ⚠️

Coverage data is based on head (99c4742) compared to base (eb677d6).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
- Coverage   77.60%   77.04%   -0.56%     
==========================================
  Files           3        3              
  Lines         125      122       -3     
  Branches       12       12              
==========================================
- Hits           97       94       -3     
  Misses         28       28              
Impacted Files Coverage Δ
playground/nuxt.config.ts 100.00% <ø> (ø)
src/runtime/components/NuxtTime.vue 72.54% <100.00%> (+0.54%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@danielroe danielroe marked this pull request as ready for review February 13, 2023 10:31
@danielroe danielroe merged commit 1ed176b into main Feb 13, 2023
@danielroe danielroe deleted the feat/reactive-time branch February 13, 2023 10:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants