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
Deprecate portfolio and analytics views #321
Conversation
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not remove those views and the links to it entirely?
Maybe just because I'm not emotionally there yet. I didn't set out to throw out code here, just to unlink that which wasn't made compatible at this point with the new data model, and is therefore essentially broken. I guess the links are decent placeholders for placing back links to whatever we end up replacing the analytics view with. And in the views there are some UI tables that could probably be moved rather than deleted, which should result in a more useful git history. |
So we can make a ticket to remove them in the deprecation ticket? |
I made a new ticket in project 11. |
No description provided.