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

Remove deprecated summary fields totalCellCount and projectEstimatedCellCount #3650

Closed
dsotirho-ucsc opened this issue Nov 18, 2021 · 2 comments
Assignees
Labels
API API change affecting callers debt [type] A defect incurring continued engineering cost demo [process] To be demonstrated at the end of the sprint demoed [process] Successfully demonstrated to team enh [type] New feature or request orange [process] Done by the Azul team service [subject] The service part of Azul

Comments

@dsotirho-ucsc
Copy link
Contributor

Remove /index/summary response fields deprecated by #3521

@dsotirho-ucsc dsotirho-ucsc added the orange [process] Done by the Azul team label Nov 18, 2021
@theathorn theathorn added API API change affecting callers debt [type] A defect incurring continued engineering cost service [subject] The service part of Azul enh [type] New feature or request labels Nov 18, 2021
@melainalegaspi
Copy link

melainalegaspi commented Dec 2, 2021

@hannes-ucsc :"Triage: why isn't this blocked by a browser ticket?"

amarjandu pushed a commit that referenced this issue Jan 21, 2022
dsotirho-ucsc added a commit that referenced this issue Jan 25, 2022
This commit adds the changes from the original PR that are missing from the stale commits below.

Stale commits (ascending order):
d4b0fec Add summary cell counts based on projects index (#3521)
fce53f0 Add FIXME (#3650)
07edd83 Remove unused canned bundle 4f2fc365
3eb4af1 Remove unused canned bundle ddb8f660
690eca9 (merge commit)
hannes-ucsc added a commit that referenced this issue Jan 25, 2022
…3781)

This commit adds the changes from the original PR that are missing from the stale commits below.

Stale commits (ascending order):
d4b0fec Add summary cell counts based on projects index (#3521)
fce53f0 Add FIXME (#3650)
07edd83 Remove unused canned bundle 4f2fc365
3eb4af1 Remove unused canned bundle ddb8f660
690eca9 (merge commit)
@hannes-ucsc
Copy link
Member

For demo, show that fields mentioned in issue title are absent from the summary response.

@theathorn theathorn added the demo [process] To be demonstrated at the end of the sprint label Apr 4, 2022
@melainalegaspi melainalegaspi added the demoed [process] Successfully demonstrated to team label Apr 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API API change affecting callers debt [type] A defect incurring continued engineering cost demo [process] To be demonstrated at the end of the sprint demoed [process] Successfully demonstrated to team enh [type] New feature or request orange [process] Done by the Azul team service [subject] The service part of Azul
Projects
None yet
Development

No branches or pull requests

4 participants