Skip to content

Conversation

@coderatomy
Copy link
Collaborator

@coderatomy coderatomy commented Nov 12, 2023

Summary

Adds login modal for non-authenticated users
Closes #802

Changes

  • Adds support for modalising the Login component
  • Non-authenticated users can only search through the featured projects

Screenshots

@coderatomy coderatomy force-pushed the search-modal branch 2 times, most recently from bbcbb8b to f14ae18 Compare November 12, 2023 08:32
@coderatomy coderatomy marked this pull request as ready for review November 12, 2023 08:37
@coderatomy coderatomy changed the title Search modal (feat) Add login modal when searching projects for non-authenticated users Nov 12, 2023
@tuxology
Copy link
Member

Hey @coderatomy, thanks for redoing this 👍 I tried to test this and was unable to get the featured projects show up on test instance (behind the modal) when I searched as non-logged in user. Here is an example:

image

The project does exist and is marked as featured project 👇

image


const mapDispatchToProps = dispatch => {
return {
getStaffPicks: args => {
Copy link
Member

Choose a reason for hiding this comment

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

I see that we are only getting staff picks, but even when staff pics are enabled, we are not rendering. What could be the issue? Can you test locally and make changes if needed?

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.

Search is broken for non-logged in users

2 participants