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

Feature: Template flows #2975

Merged
merged 28 commits into from
Jul 24, 2024
Merged

Feature: Template flows #2975

merged 28 commits into from
Jul 24, 2024

Conversation

akanshaaa19
Copy link
Member

No description provided.

@akanshaaa19 akanshaaa19 linked an issue Jul 11, 2024 that may be closed by this pull request
Copy link

github-actions bot commented Jul 11, 2024

@github-actions github-actions bot temporarily deployed to pull request July 11, 2024 06:51 Inactive
@github-actions github-actions bot temporarily deployed to pull request July 11, 2024 07:01 Inactive
Copy link

codecov bot commented Jul 11, 2024

Codecov Report

Attention: Patch coverage is 90.12346% with 8 lines in your changes missing coverage. Please review.

Project coverage is 80.15%. Comparing base (af1cb04) to head (2a6e084).

Files Patch % Lines
src/containers/Flow/FlowList/FlowList.tsx 77.41% 6 Missing and 1 partial ⚠️
src/components/floweditor/FlowEditor.tsx 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2975      +/-   ##
==========================================
+ Coverage   80.00%   80.15%   +0.15%     
==========================================
  Files         320      320              
  Lines       10236    10302      +66     
  Branches     2183     2201      +18     
==========================================
+ Hits         8189     8258      +69     
+ Misses       1398     1397       -1     
+ Partials      649      647       -2     

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

Copy link

cypress bot commented Jul 11, 2024

2 failed tests on run #5017 ↗︎

2 172 0 0 Flakiness 0

Details:

Merge 2a6e084 into af1cb04...
Project: Glific Commit: 68bb7fd3a2 ℹ️
Status: Failed Duration: 26:53 💡
Started: Jul 23, 2024 8:15 AM Ended: Jul 23, 2024 8:42 AM
Failed  cypress/e2e/flow/Flow.spec.ts • 2 failed tests

View Output

Test Artifacts
Flow > should load Flow list Test Replay Screenshots
Flow > sorting by title should work perfectly Test Replay Screenshots

Review all test suite changes for PR #2975 ↗︎

@github-actions github-actions bot temporarily deployed to pull request July 12, 2024 06:50 Inactive
@github-actions github-actions bot temporarily deployed to pull request July 12, 2024 07:12 Inactive
@github-actions github-actions bot temporarily deployed to pull request July 12, 2024 07:23 Inactive
@github-actions github-actions bot temporarily deployed to pull request July 12, 2024 07:39 Inactive
@github-actions github-actions bot temporarily deployed to pull request July 12, 2024 07:48 Inactive
@github-actions github-actions bot temporarily deployed to pull request July 12, 2024 07:59 Inactive
@akanshaaa19 akanshaaa19 requested a review from mdshamoon July 12, 2024 08:30
@github-actions github-actions bot temporarily deployed to pull request July 15, 2024 05:44 Inactive
Copy link
Member

@mdshamoon mdshamoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Ideally view it should directly open the flow. What would opening the flow in edit mode achieve.
Screenshot 2024-07-15 at 4 39 40 PM
  1. Lets call the menu item templates since we are already inside flows.
Screenshot 2024-07-15 at 4 38 33 PM

filterFlowQuery,
filterFlowQuery,
filterFlowQuery,
getFlowCountQuery(isActiveFilter),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At this point can you check why there are multiple calls being made to filter and count query and see if you can reduce them. If it seems to take more time, do it in a separate PR

@github-actions github-actions bot temporarily deployed to pull request July 15, 2024 14:55 Inactive
@github-actions github-actions bot temporarily deployed to pull request July 16, 2024 09:27 Inactive
@github-actions github-actions bot temporarily deployed to pull request July 16, 2024 09:40 Inactive
@github-actions github-actions bot temporarily deployed to pull request July 17, 2024 03:46 Inactive
@github-actions github-actions bot temporarily deployed to pull request July 17, 2024 03:54 Inactive
@github-actions github-actions bot temporarily deployed to pull request July 17, 2024 04:13 Inactive
@akanshaaa19
Copy link
Member Author

  1. Ideally view it should directly open the flow. What would opening the flow in edit mode achieve.
Screenshot 2024-07-15 at 4 39 40 PM 2. Lets call the menu item templates since we are already inside flows. Screenshot 2024-07-15 at 4 38 33 PM

on view it we will show description about the template flow thats why it takes to edit page first

@akanshaaa19 akanshaaa19 requested a review from mdshamoon July 17, 2024 04:23
@github-actions github-actions bot temporarily deployed to pull request July 18, 2024 04:59 Inactive
@github-actions github-actions bot temporarily deployed to pull request July 19, 2024 06:53 Inactive
Copy link
Member

@mdshamoon mdshamoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check failing unit and integration test cases. Otherwise looks good

@github-actions github-actions bot temporarily deployed to pull request July 23, 2024 05:17 Inactive
@github-actions github-actions bot temporarily deployed to pull request July 23, 2024 08:07 Inactive
@mdshamoon mdshamoon merged commit 396a97e into master Jul 24, 2024
9 of 13 checks passed
@mdshamoon mdshamoon deleted the feature/template-flows branch July 24, 2024 06:50
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.

Feature: Template Flows in Glific
2 participants