Skip to content

Conversation

SHIVRAJSINGHDEORA
Copy link

@SHIVRAJSINGHDEORA SHIVRAJSINGHDEORA commented Sep 21, 2025

BakeGenius.ai.-.Perfect.Baking.Every.Time.and.6.more.pages.-.Personal.-.Microsoft_.Edge.2025-09-20.17-37-23.mp4

πŸš€ Summary of Changes

  • New feature: Implemented mandatory login – now a user must log in before getting access to any page other than the landing page.
  • Fixed small issues and made minor UI adjustments on the login page.
  • Closed possible loopholes that allowed direct access to inner pages without logging in.

Related Issue: #


πŸ› οΈ Type of Change

  • [ βœ… ] Bug fix
  • [βœ… ] New feature (requires users to log in before accessing other pages)
  • [βœ… ] Improvement
  • [ βœ… ] Documentation update

βœ… How Has This Been Tested?

  • Tried opening protected pages directly via URL while logged out β†’ correctly redirected to landing page.
  • Verified login/logout flow across multiple browsers (Chrome, Edge).
  • Checked that the login modal works properly on desktop and mobile.

πŸ“‹ Checklist

  • [βœ… ] Code follows the project’s style guidelines
  • [βœ… ] Performed self-review of the code
  • [βœ… ] Added comments where necessary
  • [βœ… ] Added/updated tests to confirm the feature works
  • [βœ… ] Updated documentation where needed
BakeGenius.ai.-.AI-Powered.Baking.Conversions._.Perfect.Baking.Every.Time.and.5.more.pages.-.Personal.-.Microsoft_.Edge.2025-09-20.17-29-53.mp4

Copy link

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. πŸ™ŒπŸΌ

@supriya46788
Copy link
Owner

@SHIVRAJSINGHDEORA resolve conflicts

@SHIVRAJSINGHDEORA
Copy link
Author

@supriya46788 all conflicts are resolved

@SHIVRAJSINGHDEORA
Copy link
Author

@supriya46788 it is about 24 hours plz check the pr

@supriya46788
Copy link
Owner

@SHIVRAJSINGHDEORA please open new pr with latest fork

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.

2 participants