RepoSense icon indicating copy to clipboard operation
RepoSense copied to clipboard

URL for a dashboard is quite long when the `merge groups` is used

Open damithc opened this issue 4 years ago • 3 comments

It looks like the URL for a dashboard is quite long when the merge groups is used e.g.,

https://nus-cs2103-ay2021s2.github.io/tp-dashboard/?search=&sort=groupTitle&sortWithin=title&timeframe=commit&mergegroup=AY2021S2-CS2103-T14-1%2Ftp%5Bmaster%5D~AY2021S2-CS2103-T14-2%2Ftp%5Bmaster%5D~AY2021S2-CS2103-T14-3%2Ftp%5Bmaster%5D~AY2021S2-CS2103-T14-4%2Ftp%5Bmaster%5D~AY2021S2-CS2103-T16-1%2Ftp%5Bmaster%5D~AY2021S2-CS2103-T16-2%2Ftp%5Bmaster%5D~AY2021S2-CS2103-T16-3%2Ftp%5Bmaster%5D~AY2021S2-CS2103-W16-1%2Ftp%5Bmaster%5D~AY2021S2-CS2103-W16-2%2Ftp%5Bmaster%5D~AY2021S2-CS2103-W16-3%2Ftp%5Bmaster%5D~AY2021S2-CS2103-W16-4%2Ftp%5Bmaster%5D~AY2021S2-CS2103-W17-1%2Ftp%5Bmaster%5D~AY2021S2-CS2103-W17-2%2Ftp%5Bmaster%5D~AY2021S2-CS2103-W17-3%2Ftp%5Bmaster%5D~AY2021S2-CS2103-W17-4%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-T10-1%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-T10-2%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-T11-1%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-T11-2%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-T11-3%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-T11-4%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-T12-1%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-T12-2%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-T12-3%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-T12-4%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-T13-1%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-T13-2%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-T13-3%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-T13-4%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W10-1%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W10-2%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W10-3%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W10-4%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W12-1%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W12-2%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W12-3%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W13-1%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W13-2%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W13-3%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W13-4%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W14-1%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W14-2%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W14-3%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W14-4%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W15-1%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W15-2%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W15-3%2Ftp%5Bmaster%5D~AY2021S2-CS2103T-W15-4%2Ftp%5Bmaster%5D&groupSelect=groupByRepos&breakdown=true&checkedFileTypes=docs~functional-code~test-code~other&since=2021-02-19&tabOpen=true&tabType=authorship&tabAuthor=hengyongming&tabRepo=AY2021S2-CS2103T-W14-4%2Ftp%5Bmaster%5D&authorshipIsMergeGroup=true&isTabOnMergedGroup=true&authorshipFilesGlob=*.fxml

image

URLs have length limits. We might break such a limit if we are not careful.

damithc avatar Apr 09 '21 07:04 damithc

This is because each group can be merged or unmerged independently.

gerhean avatar Apr 10 '21 05:04 gerhean

This is because each group can be merged or unmerged independently.

I see. I guess we can optimize e.g., have two parameters merged and unmerged and use the one that gives a shorter URL, based on how many are merged. We can also use something like merged=all when applicable.

damithc avatar Apr 10 '21 06:04 damithc

Another idea is to use bitmap or the index of the repo in the report.

gerhean avatar Apr 10 '21 09:04 gerhean