[Bug]: Reference to Digital and DES should be removed

[vscaiceanu-1a]

Package name

all packages

Package version

9.5.0

Reproduction steps

Search for DES and Digital in the repository

Current result

Many references are still present

Expected result

References to Digital and DES should be removed

Additional comments

No response

[khardikk]

Hi @vscaiceanu-1a can i work on this issue? I have gone through the code, wanted to know if we remove the references to DES and Digital then will it not effect further paragraphs dependent on it. Like this one for instance.

[kpanot]

Hi @khardikk ,

Sure you can you can work on the task.
please do assign it to ourself before starting.

To reply to your question: it is depending of the link:

• if the link is targetinbg DES/Digital documentation/application that is not already part of this repository (or https://github.com/AmadeusITGroup/ngx-prefetch/), it can be removed. (Up to the reviewer of the Pull Request do decide if the targeted document can be opened later on).
• if the link is targeted old documentation as is now already part of this repository (the one targeted in your screenshot for example), this link need to be re-targeted (and link label can be also potentially be changed).

[khardikk]

Hi @khardikk ,

Sure you can you can work on the task.
please do assign it to ourself before starting.

To reply to your question: it is depending of the link:

• if the link is targetinbg DES/Digital documentation/application that is not already part of this repository (or https://github.com/AmadeusITGroup/ngx-prefetch/), it can be removed. (Up to the reviewer of the Pull Request do decide if the targeted document can be opened later on).
• if the link is targeted old documentation as is now already part of this repository (the one targeted in your screenshot for example), this link need to be re-targeted (and link label can be also potentially be changed).

Hey @kpanot thanks for letting me work on this, wanted to ask, how can I assign it to myself as I don't have the permissions.

[kpanot]

@khardikk ,
My bad, I did not realize you need the permission for it.
I have now assigned the issue to you.

[khardikk]

Hi @khardikk ,

Sure you can you can work on the task. please do assign it to ourself before starting.

To reply to your question: it is depending of the link:

• if the link is targetinbg DES/Digital documentation/application that is not already part of this repository (or https://github.com/AmadeusITGroup/ngx-prefetch/), it can be removed. (Up to the reviewer of the Pull Request do decide if the targeted document can be opened later on).
• if the link is targeted old documentation as is now already part of this repository (the one targeted in your screenshot for example), this link need to be re-targeted (and link label can be also potentially be changed).

Hi @kpanot as you mentioned, the links that are already part of this repo, need to be re-targeted, where do they need to be re-targeted exactly?

[kpanot]

Hi @khardikk ,
If a link is targeting a document that does not exist (or part of an non-accessible repository) but a documentation matching matching the link is present in this repository, then the URL of the link need to be changed to target this matching documentation.

[vscaiceanu-1a]

hi @khardikk
Don't worry that much about the links, it Will be fixed in #1267

[khardikk]

hi @khardikk
Don't worry that much about the links, it Will be fixed in #1267

Alright thanks for letting me know