Skip to content

Conversation

@Vinnl
Copy link
Contributor

@Vinnl Vinnl commented Jul 16, 2019

We can't literally inject the module, since it determines the
current user in a different way (based on who's logged in rather
than on which Pod we're running), so this is currently a copy-paste
of most code in there, with a few import paths changed around.

Still, it at least shows where to copy changes from datasister
into, and vice versa: it maintains a similar overall structure, so
modifications made here can relatively easily be carried over to
datasister.

Note that this builds on #117, which wasn't merged yet. Thus, for reviewing it's probably easiest to just focus on these changes: 760db5c

Vinnl added 4 commits June 12, 2019 11:41
We can't literally inject the module, since it determines the
current user in a different way (based on who's logged in rather
than on which Pod we're running), so this is currently a copy-paste
of most code in there, with a few import paths changed around.

Still, it at least shows where to copy changes from datasister
into, and vice versa: it maintains a similar overall structure, so
modifications made here can relatively easily be carried over to
datasister.
@megoth
Copy link
Contributor

megoth commented Aug 12, 2019

This feature was made obsolete by the introduction of global dashboard by @timbl. Closing this PR for now.

Reopen if you want to continue this work.

@megoth megoth closed this Aug 12, 2019
@timea-solid timea-solid deleted the datasister-dashboard branch February 28, 2022 17:07
@timea-solid timea-solid restored the datasister-dashboard branch February 28, 2022 17:07
@timea-solid timea-solid deleted the datasister-dashboard branch March 11, 2022 13:32
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 this pull request may close these issues.

3 participants