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

[WEB - 466] perf: improve performance for cycle and module endpoints #3711

Merged
merged 24 commits into from
Feb 21, 2024

Conversation

pablohashescobar
Copy link
Collaborator

@pablohashescobar pablohashescobar commented Feb 20, 2024

WEB - 466

perf:

  • remove unnecessary data.
  • return only success when creating issues in a cycle or module.
  • create separate endpoint for fetching issues by list of issue_ids.

Front end changes made to accommodate the perf related changes,

  • Type changes for module type, to remove details of objects like project, workspace, member, lead etc and use only the ids
  • Type changes for cycle type, to remove details of objects like project, workspace, assignees etc and use only ids
  • Changes to accommodate for handling adding issues to module and cycles

web/components/modules/delete-module-modal.tsx Dismissed Show dismissed Hide dismissed
web/components/modules/module-card-item.tsx Dismissed Show dismissed Hide dismissed
web/components/modules/module-list-item.tsx Dismissed Show dismissed Hide dismissed
@pablohashescobar pablohashescobar marked this pull request as ready for review February 20, 2024 13:36
@pablohashescobar pablohashescobar changed the title perf: improve performance for cycle and module endpoints [WEB - 466] perf: improve performance for cycle and module endpoints Feb 20, 2024
@sriramveeraghanta sriramveeraghanta merged commit ab3c3a6 into develop Feb 21, 2024
6 of 9 checks passed
@sriramveeraghanta sriramveeraghanta deleted the perf/cycle-module-endpoints branch February 21, 2024 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants