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: Allow to import time field #17677

Merged
merged 2 commits into from Aug 1, 2022

Conversation

shariquerik
Copy link
Member

Issue: Cannot import the Time field through Data Import.

Excel:
image

Before:
image

After:
image

@codecov
Copy link

codecov bot commented Jul 29, 2022

Codecov Report

Merging #17677 (12cd367) into develop (88ad1c9) will decrease coverage by 1.36%.
The diff coverage is 33.33%.

@@             Coverage Diff             @@
##           develop   #17677      +/-   ##
===========================================
- Coverage    57.25%   55.88%   -1.37%     
===========================================
  Files          751      751              
  Lines        67773    67782       +9     
  Branches      5880     5880              
===========================================
- Hits         38802    37880     -922     
- Misses       25292    26223     +931     
  Partials      3679     3679              
Flag Coverage Δ
server 62.01% <33.33%> (-2.15%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@shariquerik shariquerik merged commit c478673 into frappe:develop Aug 1, 2022
mergify bot pushed a commit that referenced this pull request Aug 1, 2022
shariquerik added a commit that referenced this pull request Aug 1, 2022
(cherry picked from commit c478673)

Co-authored-by: Shariq Ansari <30859809+shariquerik@users.noreply.github.com>
frappe-pr-bot pushed a commit that referenced this pull request Aug 9, 2022
# [13.37.0](v13.36.3...v13.37.0) (2022-08-09)

### Bug Fixes

* allow system managers to toggle email queue ([c004846](c004846))
* allow to import time field ([#17677](#17677)) ([#17690](#17690)) ([59f58c7](59f58c7))
* check permission before sending email ([bb73014](bb73014))
* Dropdown selection list in Reports should be translatable ([#17679](#17679)) ([8c0e5ab](8c0e5ab))
* **global_search:** Trigger rebuilding on Custom Field's property change ([#17705](#17705)) ([b4ff5b2](b4ff5b2))
* max_positive_value for Integer types ([#17712](#17712)) ([#17713](#17713)) ([e173e64](e173e64))
* merge conflict ([2f71fb1](2f71fb1))
* restored patch and remove unused import ([b2232f1](b2232f1))
* return promise ([#17646](#17646)) ([#17666](#17666)) ([d931e26](d931e26))
* Show Report & Dashboard View for File Doctype (backport [#17688](#17688)) ([#17695](#17695)) ([ad09e87](ad09e87))
* version bump regex in automated release ([#17725](#17725)) ([#17726](#17726)) ([a4f682a](a4f682a))

### Features

* Role based permission for Dashboard Chart ([#17634](#17634)) ([#17655](#17655)) ([c3f45e7](c3f45e7))

### Performance Improvements

* reduce DB calls in `frappe.client.get` (and other changes) (backport [#17665](#17665)) ([#17710](#17710)) ([5fe4924](5fe4924))
* reduce DB calls made in `get_fetch_values` ([#17671](#17671)) ([#17738](#17738)) ([fc58a87](fc58a87))
* specify reference doctype in filters ([#17703](#17703)) ([88a59d8](88a59d8))
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant