sourcegraph / sourcegraph-public-snapshot

Code AI platform with Code Search & Cody
https://sourcegraph.com
Other
10.1k stars 1.27k forks source link

Insights placeholder: dashboard keep order and size #29960

Open Joelkw opened 2 years ago

Joelkw commented 2 years ago

Local order and size persistence was implemented by https://github.com/sourcegraph/sourcegraph/pull/50301

A remaining follow-up for this problem is adding a remote sync logic to keep size and order in public dashboards for more than one user

github-actions[bot] commented 2 years ago

Heads up @joelkw @felixfbecker @vovakulikov @unclejustin - the "team/code-insights" label was applied to this issue.

Joelkw commented 2 years ago

Initial spike at #29536

Joelkw commented 2 years ago

Current status: in backlog; this is significant work that will be made easier by some earlier roadmap items first!

felixfbecker commented 2 years ago

Which other earlier roadmap items are you referring to?

Joelkw commented 2 years ago

The API changes that we're making to adapt to new insight types, as well as having more onboarded backend folks who can help out with this.

r-shahid commented 1 year ago

+1 from this customer: https://github.com/sourcegraph/accounts/issues/7229

nihaals commented 1 year ago

Is there another issue tracking what #50301 didn't include (remote storage)?

vovakulikov commented 1 year ago

Sorry for the slow response. To my knowledge, we don't have a designated issue for the remote storage part. I will reopen this issue and add a short description of remote size and order syncing. Thanks for noticing.

nihaals commented 8 months ago

@vovakulikov Was this completed? I can't a corresponding PR or changelog entry.

vovakulikov commented 8 months ago

Hello @nihaals, sorry, I actually miss-clicked this, the current state, local storage solution was done here https://github.com/sourcegraph/sourcegraph/pull/50301, the remote store isn't done and I don't think will be in a near future. I reopen this one until we will do the remote store solution.