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 issue 186 #305

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix issue 186 #305

wants to merge 2 commits into from

Conversation

alenap93
Copy link
Contributor

@alenap93 alenap93 commented Feb 6, 2025

Description

Fixed error 400 'The value must be a valid' when a field of type timestamp is added or edited. Furthermore added unit tests to TimestampInputComponent

Related Issues

#186

How can it be tested?

Check list before submitting

  • This PR is wrote in a clear language and correctly labeled
  • I created the related changeset for my changes with npx changeset
  • I have performed a self-review of my code (no debugs, no commented code, good naming, etc.)
  • I wrote the relative tests
  • I created a PR for the documentation if necessary and attached the link to this PR

…ield of type timestamp is added or edited. Furthermore added unit tests to TimestampInputComponent
Copy link

codecov bot commented Feb 6, 2025

Bundle Report

Bundle size has no change ✅

Copy link

codecov bot commented Feb 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.60%. Comparing base (296a571) to head (79eb9af).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #305      +/-   ##
==========================================
+ Coverage   65.40%   65.60%   +0.20%     
==========================================
  Files          51       51              
  Lines        1451     1451              
  Branches      331      331              
==========================================
+ Hits          949      952       +3     
- Misses        461      497      +36     
+ Partials       41        2      -39     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

1 participant