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

Implement the functionality of UpdateProfilePendingOwner #31

Merged
merged 6 commits into from
Aug 5, 2024

Conversation

VictoriaAde
Copy link
Contributor

This PR implements the update_profile_pending_owner function.
The function updates the "pending owner" of a specific profile. The "pending owner" is someone who is set to become the new owner of the profile.

Fixes #9

@VictoriaAde VictoriaAde marked this pull request as ready for review June 25, 2024 23:00
@VictoriaAde
Copy link
Contributor Author

@Akashneelesh I am done with the task, please take a look.

@Akashneelesh
Copy link
Collaborator

Hey @VictoriaAde , could you please resolve the merge conflicts.

@VictoriaAde
Copy link
Contributor Author

Hey @VictoriaAde , could you please resolve the merge conflicts.

On it! I'm currently encountering crazy bugs, and fixing them atm.

@VictoriaAde
Copy link
Contributor Author

Hi @Akashneelesh, I have fixed the conflicts and used the _is_owner_of_profile in my function to check for ownership.

@Akashneelesh Akashneelesh merged commit 135a327 into keep-starknet-strange:main Aug 5, 2024
0 of 2 checks passed
@mikoU77
Copy link

mikoU77 commented Aug 6, 2024

good

@Akashneelesh
Copy link
Collaborator

@VictoriaAde The reward has been sent out. LFG

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.

[feat] Implement the functionality of UpdateProfilePendingOwner
3 participants