Skip to content

feature/agile-60 provide created vs. resolved chart - #24862

Open
AileenK13 wants to merge 17 commits into
opf:devfrom
AileenK13:feature/agile-60-provide-created-vs-resolved-chart
Open

AileenK13 wants to merge 17 commits into
opf:devfrom
AileenK13:feature/agile-60-provide-created-vs-resolved-chart

Conversation

@AileenK13

@AileenK13 AileenK13 commented Aug 20, 2026 •

Copy link
Copy Markdown

Ticket

https://community.openproject.org/wp/AGILE-60

What are you trying to accomplish?

Creating a “Created vs. Resolved” Graph for the Sprint Report

Screenshots

2026-08-20_19h10_17

What approach did you choose and why?

Based on the burndown chart, another widget was developed that displays the tasks created and completed during a sprint.

AI involvement

None/Assisted – No AI assistance used OR only autocomplete/pasted snippets. I effectively wrote and understand all the code.

Merge checklist

  • Added/updated tests
  • Added/updated documentation in Lookbook (patterns, previews, etc)
  • Tested major browsers (Chrome, Firefox, Edge, ...)

@github-actions

github-actions Bot commented Aug 20, 2026 •

Copy link
Copy Markdown

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@AileenK13

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

@EinLama
EinLama self-requested a review August 25, 2026 13:04
Comment thread modules/backlogs/app/models/created_resolved.rb Fixed

@EinLama EinLama left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey @AileenK13 , thank you for this PR. It renders a useful chart and already has a lot going for it 👏 well done!

Before we can merge it, there is some feedback that should be addressed. Also, some tests (we call them specs) should be added. I know that this is already on your todo-list, just listing it here for the sake of completeness.

The CI uncovered some issues during an automatic check (linting, style, etc.), please have a look at these, too. I didn't add an additional review comment for most of these and let them stand for themselves.

I know that this is a lot of feedback. Please inquire further when something is unclear or you require support 👍

Thank you for your work 🙇

Comment thread modules/backlogs/lib/open_project/backlogs/created_resolved/series.rb Outdated
Comment thread modules/backlogs/app/models/created_resolved.rb Outdated
Comment thread modules/backlogs/app/models/created_resolved.rb Outdated
Comment thread modules/backlogs/app/models/created_resolved.rb Outdated
Comment thread modules/backlogs/app/models/created_resolved.rb Outdated
Comment thread frontend/src/app/shared/components/work-package-graphs/plugin.primer-colors.ts Outdated
Comment thread modules/backlogs/lib/open_project/backlogs/created_resolved/series_raw_data.rb Outdated
Comment thread modules/backlogs/app/models/created_resolved.rb Outdated
Comment thread frontend/src/app/features/backlogs/created-resolved-chart.component.ts Outdated
Comment thread modules/backlogs/lib/open_project/backlogs/created_resolved/series_raw_data.rb Outdated

@EinLama EinLama left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There are some TypeScript linting errors in the chart component, you can run npx eslint <file> locally to see them. For example, there are some semicolons missing. Please have a look and fix them 🧹

Comment thread frontend/src/app/features/backlogs/created-resolved-chart.component.ts Outdated
Comment thread frontend/src/app/features/backlogs/created-resolved-chart.component.ts Outdated
Comment thread frontend/src/app/features/backlogs/created-resolved-chart.component.ts Outdated
Comment thread frontend/src/app/features/backlogs/created-resolved-chart.component.ts Outdated
ORDER BY days.date
SQL

Journal::WorkPackageJournal.connection.select_all(query_string)
@AileenK13 AileenK13 changed the title Erstellung Created vs resolved Chart Creating Created vs resolved Chart Sep 14, 2026
@AileenK13 AileenK13 changed the title Creating Created vs resolved Chart provide created vs. resolved chart Sep 14, 2026
@AileenK13 AileenK13 changed the title provide created vs. resolved chart feature/agile-60 provide created vs. resolved chart Sep 14, 2026
@github-actions github-actions Bot added the ai: None/Assisted 👤 No AI assistance used OR only snippets. A human effectively wrote and understands all the code. label Oct 10, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai: None/Assisted 👤 No AI assistance used OR only snippets. A human effectively wrote and understands all the code. contribution

Development

Successfully merging this pull request may close these issues.

6 participants