add import of missing Angular interfaces

[saschaszott]

Description

This PR adds imports of missing Angular interfaces, e.g. OnInit, OnDestroy etc.

@tdonohue , is it possible to add an appropriate lint check?

[saschaszott]

@tdonohue , maybe the best way of handling this issue (missing imports of Angular lifecycle interfaces) in the ongoing development efforts is the adoption of an additional lint rule as suggested by @ybnd .

[alanorth]

Small nitpick, could this be rebased and squashed into one commit to keep our git log neat and tidy? Thanks!

[saschaszott]

Small nitpick, could this be rebased and squashed into one commit to keep our git log neat and tidy? Thanks!

@alanorth should I open a new PR or do you plan to squash all the commits at merge time?

[alanorth]

Hi @saschaszott, we could rebase/squash on GitHub when merging if we remember, but it would be easy for you to do in your branch as well. Assuming you have a git remote for the upstream DSpace repository you could do:

$ git rebase -i upstream/dspace-7_x

Then you will get a list of commits with actions to be performed. You change everything after the first one from pick to squash and git will combine them all into the first commit and let you enter a new commit message after saving:

pick 6dc2ea3 added missing interfaces
squash 8bce1d8 added missing OnInit interface                                                                                                          
squash fee794b added missing interface OnInit
...

[saschaszott]

Hi @saschaszott, we could rebase/squash on GitHub when merging if we remember, but it would be easy for you to do in your branch as well. Assuming you have a git remote for the upstream DSpace repository you could do:

$ git rebase -i upstream/dspace-7_x

Then you will get a list of commits with actions to be performed. You change everything after the first one from pick to squash and git will combine them all into the first commit and let you enter a new commit message after saving:

pick 6dc2ea3 added missing interfaces
squash 8bce1d8 added missing OnInit interface                                                                                                          
squash fee794b added missing interface OnInit
...

@alanorth , this does not work as expected. I get tons of merge conflicts. Any ideas how to handle this situation?

[alanorth]

@saschaszott Ah, in my comment I suggested rebasing on top of dspace-7_x, but your pull request is against main so you should rebase (and squash) on top of that instead. If your commits apply clean, then a squashed version of the commits should as well (the only difference is one large commit instead of many small ones).

[saschaszott]

@alanorth , thank you. This does not work either - I'm lost in merge conflicts after running git rebase -i upstream main. I will try another approach and create a new PR (with one commit).

[saschaszott]

@alanorth , @tdonohue : you can find the squashed version of this PR in #3048 - I'll close this PR