Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename addUser method to addUniqueIdToUser in UserUniqueIDManger #22243

Closed
wants to merge 1 commit into from

Conversation

Sachin-Mamoru
Copy link
Contributor

@Sachin-Mamoru Sachin-Mamoru commented Jan 14, 2025

Related to #22086

Rename the addUser method in UserUniqueIDManger to addUniqueIdToUser and deprecate the original method.

  • UserUniqueIDManger.java

    • Rename the public method addUser to addUniqueIdToUser.
    • Add a @Deprecated annotation to the original addUser method.
    • Update the Javadoc comment to reflect the new method name.
  • UserManagementClient.java

    • Update the call to addUser to addUniqueIdToUser.
  • UUIDUserStoreManagerService.java

    • Update the call to addUser to addUniqueIdToUser.

For more details, open the Copilot Workspace session.

Related to wso2#22086

Rename the `addUser` method in `UserUniqueIDManger` to `addUniqueIdToUser` and deprecate the original method.

* **UserUniqueIDManger.java**
  - Rename the public method `addUser` to `addUniqueIdToUser`.
  - Add a `@Deprecated` annotation to the original `addUser` method.
  - Update the Javadoc comment to reflect the new method name.

* **UserManagementClient.java**
  - Update the call to `addUser` to `addUniqueIdToUser`.

* **UUIDUserStoreManagerService.java**
  - Update the call to `addUser` to `addUniqueIdToUser`.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/wso2/product-is/issues/22086?shareId=XXXX-XXXX-XXXX-XXXX).
@Sachin-Mamoru
Copy link
Contributor Author

Test PR
Please Ignore

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.

1 participant