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

bug: completion rate is invalid for filecoin-station/roadmap#15 #360

Closed
SgtPooki opened this issue May 4, 2023 · 4 comments · Fixed by #361
Closed

bug: completion rate is invalid for filecoin-station/roadmap#15 #360

SgtPooki opened this issue May 4, 2023 · 4 comments · Fixed by #361
Assignees

Comments

@SgtPooki
Copy link
Contributor

SgtPooki commented May 4, 2023

screenshot of issue in github:

image

resulting JSON when loading locally:

{
  "labels": [
    "station core"
  ],
  "completion_rate": 0,
  "due_date": "2023-05-01",
  "html_url": "https://github.com/filecoin-station/roadmap/issues/15",
  "group": "",
  "title": "Station Core 👉 Station Desktop integration",
  "state": "open",
  "node_id": "I_kwDOIUF5GM5fK-t-",
  "parent": {
    "state": "open",
    "group": "children:",
    "title": "Roadmap: Retrieval Markets",
    "html_url": "https://github.com/retrieval-markets-lab/roadmap/issues/1",
    "labels": [
      "starmaps",
      "roadmap"
    ],
    "node_id": "I_kwDOIhfhEc5XfRvq",
    "completion_rate": 0,
    "due_date": "",
    "description": ""
  },
  "children": [],
  "description": "Greatly simplify `Station Desktop` and the product ecosystem in general by letting it leverage `Station Core`."
}
@SgtPooki
Copy link
Contributor Author

SgtPooki commented May 4, 2023

The black text in the rendered milestone show that no children were recognized:

image

@SgtPooki
Copy link
Contributor Author

SgtPooki commented May 4, 2023

The value of result for space-meridian/roadmap#15 from

const result: GithubIssueData = {
html_url: data.html_url,
title: data.title,
state: data.state as IssueStates,
node_id: data.node_id,
body_html: data.body_html || '',
body: data.body || '',
labels: data.labels
.map((label) => (typeof label !== 'string' ? label.name : label)) as string[],
description
};
is below:

{
  "html_url": "https://github.com/filecoin-station/roadmap/issues/15",
  "title": "Station Core 👉 Station Desktop integration",
  "state": "open",
  "node_id": "I_kwDOIUF5GM5fK-t-",
  "body_html": "<p dir=\"auto\">ETA: 2023-05-01</p>\n<p dir=\"auto\">description: Greatly simplify <code class=\"notranslate\">Station Desktop</code> and the product ecosystem in general by letting it leverage <code class=\"notranslate\">Station Core</code>.</p>\n<p dir=\"auto\">Children:</p>\n<ul class=\"contains-task-list\">\n<li class=\"task-list-item\"><input type=\"checkbox\" id=\"\" disabled=\"\" class=\"task-list-item-checkbox\" checked=\"\"> <a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"1653584777\" data-permission-text=\"Title is private\" data-url=\"https://github.com/filecoin-station/core/issues/83\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/filecoin-station/core/pull/83/hovercard\" href=\"https://github.com/filecoin-station/core/pull/83\">filecoin-station/core#83</a></li>\n<li class=\"task-list-item\"><input type=\"checkbox\" id=\"\" disabled=\"\" class=\"task-list-item-checkbox\" checked=\"\"> <a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"1676397485\" data-permission-text=\"Title is private\" data-url=\"https://github.com/filecoin-station/core/issues/100\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/filecoin-station/core/pull/100/hovercard\" href=\"https://github.com/filecoin-station/core/pull/100\">filecoin-station/core#100</a></li>\n<li class=\"task-list-item\"><input type=\"checkbox\" id=\"\" disabled=\"\" class=\"task-list-item-checkbox\"> <a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"1655290530\" data-permission-text=\"Title is private\" data-url=\"https://github.com/filecoin-station/desktop/issues/621\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/filecoin-station/desktop/pull/621/hovercard\" href=\"https://github.com/filecoin-station/desktop/pull/621\">filecoin-station/desktop#621</a></li>\n</ul>\n<p dir=\"auto\">Milestone: <a href=\"https://github.com/filecoin-station/core/milestone/4\">https://github.com/filecoin-station/core/milestone/4</a></p>",
  "body": "ETA: 2023-05-01\r\n\r\ndescription: Greatly simplify `Station Desktop` and the product ecosystem in general by letting it leverage `Station Core`.\r\n\r\nChildren:\r\n- [x] https://github.com/filecoin-station/core/pull/83\r\n- [x] https://github.com/filecoin-station/core/pull/100\r\n- [ ] https://github.com/filecoin-station/desktop/pull/621\r\n\r\nMilestone: https://github.com/filecoin-station/core/milestone/4",
  "labels": [
    "station core"
  ],
  "description": "Greatly simplify `Station Desktop` and the product ecosystem in general by letting it leverage `Station Core`."
}

@SgtPooki
Copy link
Contributor Author

SgtPooki commented May 4, 2023

It looks like the children are in an invalid format:

const body_text = 'Children:\r\n' +
            '- [x] https://github.com/filecoin-station/core/pull/83\r\n' +
            '- [x] https://github.com/filecoin-station/core/pull/100\r\n' +
            '- [ ] https://github.com/filecoin-station/desktop/pull/621\r\n' +
            '\r\n' + // ...

see https://github.com/pln-planning-tools/Starmap/blob/main/User%20Guide.md#children-syntax (deprecated)
and https://github.com/pln-planning-tools/Starmap/blob/main/User%20Guide.md#tasklist-syntax

SgtPooki added a commit that referenced this issue May 4, 2023
* Children can be PRs now
* Correctly check for case-insensitive "children:"

fixes #360
@SgtPooki SgtPooki linked a pull request May 4, 2023 that will close this issue
@juliangruber
Copy link
Contributor

Raw markdown of the issue:

ETA: 2023-05-01

description: Greatly simplify `Station Desktop` and the product ecosystem in general by letting it leverage `Station Core`.

Children:
- [x] https://github.com/filecoin-station/core/pull/83
- [x] https://github.com/filecoin-station/core/pull/100
- [ ] https://github.com/filecoin-station/desktop/pull/621

Milestone: https://github.com/filecoin-station/core/milestone/4

@SgtPooki SgtPooki self-assigned this May 5, 2023
SgtPooki added a commit that referenced this issue May 9, 2023
* fix: children case-sensitivity and PR support

* Children can be PRs now
* Correctly check for case-insensitive "children:"

fixes #360

* Update tests/fixtures/getIssueResult/filecoin-station-roadmap-15.json

Co-authored-by: Nishant Arora <1895906+whizzzkid@users.noreply.github.com>

---------

Co-authored-by: Nishant Arora <1895906+whizzzkid@users.noreply.github.com>
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 a pull request may close this issue.

2 participants