Templates copy

[kurund]

Fixes for #2420

[github-actions]

🚀 Deployed on https://deploy-preview-2618--fervent-hypatia-766178.netlify.app

[cypress]

3 failed and 1 flaky tests on run #3787 ↗︎

3

172

0

0

1

Details:

Merge a50f003 into 34ef64c...
Project: Glific	Commit: 66f3cb596e ℹ️
Status: Failed	Duration: 24:10 💡
Started: Nov 29, 2023 12:44 AM	Ended: Nov 29, 2023 1:08 AM

  roles/staff/chat/Chat.spec.ts • 1 failed test

View Output

Test		Artifacts
Role - Staff - Chats > should go to top		Screenshots

  template/SpeedSend.spec.ts • 2 failed tests

View Output

Test		Artifacts
Speed Send > should edit speed send		Screenshots
Speed Send > should show all languages		Screenshots

  cypress/e2e/staffmanagement/StaffManagement.spec.ts • 1 flaky test

View Output

Test		Artifacts
Staff Management > should delete collection		Screenshots

Review all test suite changes for PR #2618 ↗︎

[codecov]

Codecov Report

Attention: 45 lines in your changes are missing coverage. Please review.

Comparison is base (08e6b9b) 73.16% compared to head (a50f003) 73.19%.
Report is 2 commits behind head on master.

Files	Patch %	Lines
src/containers/Template/Form/Template.tsx	47.43%	33 Missing and 8 partials ⚠️
src/containers/Template/List/Template.tsx	42.85%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2618      +/-   ##
==========================================
+ Coverage   73.16%   73.19%   +0.03%     
==========================================
  Files         242      242              
  Lines        8459     8454       -5     
  Branches     1803     1795       -8     
==========================================
- Hits         6189     6188       -1     
+ Misses       1663     1662       -1     
+ Partials      607      604       -3     

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

[kurund]

@mdshamoon

Unit tests are working fine on my local. It's failing due to an error specific to CI which you mentioned during our call.

Cypress error is fixed in glific/cypress-testing#167

We would also need to update the CI on the frontend after we merge this and Cypress PR. I can do that later.

[mdshamoon]

Overall code looks good.
When I copy a template I am not able to edit it. Ideally after copy I should be able to edit it.

[kurund]

When I copy a template I am not able to edit it. Ideally after copy I should be able to edit it.

This should be fixed.