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

Done with changes #1646 and #1644 #1808

Closed
wants to merge 0 commits into from

Conversation

Ashmitha2606
Copy link
Contributor

Pull Request Checklist

issues: #1646 and #1644
profile changes done with leading blog page change
Screenshot 2024-11-03 173424

  • I have added screenshots and videos to show before and after the working of my code.
  • I have ensured that the screen size is set to 100% while making the video.
  • I have synced the latest fork with my local repository and resolved any conflicts.
  • I have mentioned the issue number which I created before making this PR .(format to mention issue number is : fixes #(issue number) )
  • I understand that if any the above conditions are not met , my PR will not be MERGED .

Screenshot 2024-11-04 161157
Screenshot 2024-11-04 161146

Copy link

vercel bot commented Nov 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
word-wise ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 9, 2024 11:20am

@Ashmitha2606
Copy link
Contributor Author

Ashmitha2606 commented Nov 9, 2024

@ANSHIKA-26 check..please view the latest .. development..
i have closed the previous pr in and created a new one please reassign the labels

@ANSHIKA-26
Copy link
Owner

@Ashmitha2606
on clicking profile our current login looks like -
image

whereas in your preview it has removed the close and dont have an account , signup here button ,
same issue with signup page
kindly preview your changes and match with current website , make sure that while adding things you are not removing the already existing functionalities of the website !

@Ashmitha2606
Copy link
Contributor Author

Ashmitha2606 commented Nov 9, 2024

@ANSHIKA-26 ...Done with changes!!
review it

@Ashmitha2606
Copy link
Contributor Author

@Ashmitha2606 on clicking profile our current login looks like - image

whereas in your preview it has removed the close and dont have an account , signup here button , same issue with signup page kindly preview your changes and match with current website , make sure that while adding things you are not removing the already existing functionalities of the website !

ya since i have old fork of ur website the form and sign page at that time was without close and signup features...So now i have matched that to ur current website

@ANSHIKA-26
Copy link
Owner

sync fork again , the profile section has been removed

@Ashmitha2606
Copy link
Contributor Author

sync fork again , the profile section has been removed

@ANSHIKA-26 check it i have sync again

@Ashmitha2606
Copy link
Contributor Author

Ashmitha2606 commented Nov 9, 2024

Screenshot 2024-11-09 151028
@ANSHIKA-26 view the latest preview
every page has profile section

@ANSHIKA-26
Copy link
Owner

thats the problem , we removed the profile section ! someone is working on it !

@Ashmitha2606
Copy link
Contributor Author

Ashmitha2606 commented Nov 9, 2024 via email

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

Successfully merging this pull request may close these issues.

2 participants