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

bug fix for the profile page view Profile bug fix for issue #912 #928

Closed
wants to merge 3 commits into from

Conversation

biplavmz
Copy link
Contributor

on profile page we need to hide the three dot bug fix for #912

Before :-
image

After:-
image

Copy link

@github-actions github-actions bot 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 biplavmz! for creating this pull request and contributing to Dummygram! 💗

The maintainers will review this Pull Request and provide feedback as soon as possible! 😇
We appreciate your patience and contribution, Keep up the great work! 😀

Copy link
Owner

@narayan954 narayan954 left a comment

Choose a reason for hiding this comment

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

check comment

@@ -189,6 +189,8 @@ function Post(prop) {
}}
>
<ErrorBoundary>
{
!profile &&
Copy link
Owner

Choose a reason for hiding this comment

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

we don't need to hide the header completely, we just had to hide the visit profile option, as you had mentioned there as well. Also, instead of prop drilling, you can instead check the current route as well, uselocation... or there might be other better ways as well

@narayan954
Copy link
Owner

why duplicate pr? closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

2 participants