otter
otter copied to clipboard
[Bug]: Reference to Digital and DES should be removed
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
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.
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 @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.
@khardikk , My bad, I did not realize you need the permission for it. I have now assigned the issue to you.
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?
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.
hi @khardikk Don't worry that much about the links, it Will be fixed in https://github.com/AmadeusITGroup/otter/pull/1267
hi @khardikk Don't worry that much about the links, it Will be fixed in https://github.com/AmadeusITGroup/otter/pull/1267
Alright thanks for letting me know