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

Profile Update Feature #525

Merged
merged 3 commits into from
Dec 15, 2024
Merged

Conversation

Pqrq
Copy link
Contributor

@Pqrq Pqrq commented Dec 15, 2024

PR Description: Fix Profile Update Mechanism and UI Enhancements

Summary

This PR introduces fixes and improvements to the Profile Update Mechanism and enhances user interaction features across the app:

  1. Profile Update Fix:

    • Resolved issues with the profile update mechanism, ensuring seamless updates to user information, including name, bio, user tag.
    • Error messages are now displayed correctly for invalid or failed updates.
  2. Refactored Code:

    • Streamlined the update mechanism by introducing modular methods for easier maintainability.
    • Reduced redundancy in the user profile update logic.
    • Changed some of our mock services with their real versions.

Changes

  • Refactored the profile updating logic to ensure consistent updates across all fields.
  • Improved error handling and validation in both frontend and backend functions.
  • Turned some mock services to real ones, in order to align with the latest working version of the app.

User profile can now be updated
@Pqrq Pqrq added Enhancement New feature or request Priority: High Requires taking action ASAP Action Item A specific task requiring attention Frontend Mobile mobile app development labels Dec 15, 2024
Copy link
Contributor

Choose a reason for hiding this comment

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

Delete this please if not used anywhere

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, I have deleted that page as it is unnecessary.

Copy link
Contributor

@karatashsyn karatashsyn left a comment

Choose a reason for hiding this comment

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

Thank you for your efforts 🎉

@Pqrq Pqrq merged commit 16b7085 into mobile-main-updated Dec 15, 2024
@Pqrq Pqrq self-assigned this Dec 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Action Item A specific task requiring attention Enhancement New feature or request Frontend Mobile mobile app development Priority: High Requires taking action ASAP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants