Skip to content

204 mentor profiling - #213

Open
betigirma wants to merge 17 commits into
release-02from
204-mentor-profiling
Open

betigirma wants to merge 17 commits into
release-02from
204-mentor-profiling

Conversation

@betigirma

Copy link
Copy Markdown
Collaborator

No description provided.

@betigirma betigirma added the enhancement New feature or request label Mar 31, 2025
@betigirma
betigirma requested a review from misudesu March 31, 2025 16:43
@betigirma betigirma self-assigned this Mar 31, 2025
@betigirma betigirma linked an issue Mar 31, 2025 that may be closed by this pull request
@betigirma

Copy link
Copy Markdown
Collaborator Author

sorry i can't add image because of devtunnel

Comment thread actions/admin/admin.ts Outdated
return await patchRequest.patchData();
};

export const toggleMentorStatus = async (

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

put types in type folder.

Comment thread base-api/method.ts
super(url, tag, data, 'PATCH');
}

public async putData() {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

why you change the put to patch , instead of adding it? for such case we don't change such function , if we need additional we have to add as new.

@betigirma
betigirma requested a review from misudesu April 14, 2025 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Review

Development

Successfully merging this pull request may close these issues.

mentor profiling

2 participants