Skip to content

Admin delete user#15

Closed
YehiaFarghaly wants to merge 1 commit intoadmin-activationfrom
admin-delete-user
Closed

Admin delete user#15
YehiaFarghaly wants to merge 1 commit intoadmin-activationfrom
admin-delete-user

Conversation

@YehiaFarghaly
Copy link
Copy Markdown
Contributor

No description provided.

@YehiaFarghaly YehiaFarghaly requested a review from a team May 2, 2025 17:13
Copy link
Copy Markdown
Contributor

@NourAlPha NourAlPha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added 2 comments.

Comment thread logs/app.log
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here, please remove all the log files.

public class AdminController {

private final AdminService adminService;
private static final Logger logger = LoggerFactory.getLogger(AdminController.class);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same for logger.

@YehiaFarghaly YehiaFarghaly changed the base branch from dev to admin-activation May 2, 2025 19:29
@YehiaFarghaly YehiaFarghaly deleted the admin-delete-user branch May 3, 2025 12:24
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.

2 participants