Thanks for opening this issue. A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

👋 @jgimenez 🙂 To answer your main question:

It's not clear to me if there's a way to move the users to the new app, so they don't need to re-authenticate (I guess there is not, but I think should be indicated)

IIUC, no, this isn't possible. The new app is essentially a distinct installation, so it's access must be authenticated by the user.

It's not clear to me whether I should delete the old OAuth app. It's confusing to me, because it says we should encourage users to de-authorize the old app, but if the old app is no longer in use, it would make more sense to me to delete the app, so its credentials can not be abused in any way and all authorizations are revoked.

I think this was originally written implying that some users might still be using the old app. If all your users have transitioned off the old app, or if you're happy to delete the old app even though it might still have active users, then deleting the app makes sense 👍

We can definitely update the doc to be more clear about both these points. I'll open this issue up for anyone to take on 🙂

Hi, I would like to work on this issue, can anyone suggest me to get started?

👋 @prsmahajan You're welcome to open a PR following the plan outlined in #1236 (comment). Let us know if you have any additional questions!

What is the issue @janiceilene ?

What is the issue @janiceilene ?

I'm sorry @prsmahajan, I'm not sure what you're question is. If you'd like to work on this issue, you're welcome to open a pull request following the instructions given in this comment.

Please let me know if there's additional information or clarification I can give 💛

I think this was originally written implying that some users might still be using the old app. If all your users have transitioned off the old app, or if you're happy to delete the old app even though it might still have active users, then deleting the app makes sense +1

We can definitely update the doc to be more clear about both these points. I'll open this issue up for anyone to take on slightly_smiling_face

I failed to find an issue for the modifications suggested so I opened this issue and made changes in the linked pull request