Skip to content

Us6 profile#28

Merged
Ale-Coeto merged 15 commits intomainfrom
US6_Profile
Jun 28, 2025
Merged

Us6 profile#28
Ale-Coeto merged 15 commits intomainfrom
US6_Profile

Conversation

@EfrainH2O
Copy link
Copy Markdown
Contributor

@EfrainH2O EfrainH2O commented Jun 17, 2025

The route to profile infromation and edition of the leetcode user has been created.
The frontend for this routers is also added as a pop up at the User navbar.
This pop up also added has the sign out button

The router profileRouter handles the petitions for user information as well as to update the info of this attribute.
Copy link
Copy Markdown
Member

@Oscar-gg Oscar-gg left a comment

Choose a reason for hiding this comment

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

lgtm, just add awaits


export const profileRouter = createTRPCRouter({
getById: protectedProcedure.input(z.string()).query(async ({ ctx, input }) => {
return ctx.db.user.findUnique({ where: { id: input }, include: { accounts: true, sessions: false, solvedProblems: true } });
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

add an await so that you don't return a promise

}))
.mutation(async ({ ctx, input }) => {
const { id, ...data } = input;
return ctx.db.user.update({ where: { id }, data });
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

also add an await here

EfrainH2O and others added 7 commits June 17, 2025 15:57
Now there is a Profile Card to use.
It needs to be done a pop up when it is clicked the user.
In case the it can't be done, it just needs to be linked the actual page with the click on the profile
Copy link
Copy Markdown
Contributor

@GilMM27 GilMM27 left a comment

Choose a reason for hiding this comment

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

Fix build and pr description

@Ale-Coeto Ale-Coeto merged commit 3f96fa3 into main Jun 28, 2025
3 checks passed
@Rodrogamby Rodrogamby deleted the US6_Profile branch July 13, 2025 19:51
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.

5 participants