Skip to content
This repository has been archived by the owner on Oct 11, 2022. It is now read-only.

Splash page marketing update #1400

Merged
merged 30 commits into from
Aug 31, 2017
Merged

Splash page marketing update #1400

merged 30 commits into from
Aug 31, 2017

Conversation

superbryntendo
Copy link
Contributor

No description provided.

@superbryntendo superbryntendo changed the title draft content for homepage Splash page marketing update Aug 22, 2017
@superbryntendo
Copy link
Contributor Author

@mxstbr I'll need your help with wrapping this up.

I needed to move the splash page inside the router to be able to link to internal pages from both buttons AND the search component (so tags were out of the question...)

Now, the problem is that I can't force the nav bar not to render in the splash page. Brian and I both tried some solutions to this problem and came up short. Would love your help on this so we can push it out the door finally!

@mxstbr
Copy link
Contributor

mxstbr commented Aug 29, 2017

On it

@mxstbr
Copy link
Contributor

mxstbr commented Aug 29, 2017

I reworked the way the src/index.js and the router work on #1394 completely due to the constraints around being able to import the routes from Node. I'm going to move those changes into this PR manually otherwise we'll end up with a super disgusting conflict, then fix the navbar!

@mxstbr
Copy link
Contributor

mxstbr commented Aug 29, 2017

BUG: Clicking "Sign in" on the home redirects to /login, when you login from there it redirects back to /login which doesn't make much sense 😜

@mxstbr
Copy link
Contributor

mxstbr commented Aug 29, 2017

The Navbar hiding should be done, can you double check all edge cases? I think it's correct though.

@mxstbr
Copy link
Contributor

mxstbr commented Aug 29, 2017

Fixed login page in 2a2a76b!

@superbryntendo
Copy link
Contributor Author

This is ready to go IMO! Let's ship it @brianlovin!

@brianlovin
Copy link
Contributor

Logging out doesn't work for me - flashes the splash page then immediately re-renders the Dashboard. I know you were having issues with this too, @uberbryn - maybe @mxstbr can help us pinpoint why logging out isn't working?

@mxstbr
Copy link
Contributor

mxstbr commented Aug 30, 2017

Yep, this is a symptome of our logout process not working and the way we do the homepage fallback continuously now rather than determining it at first render. (like we did before)

Going to dig into this.

@superbryntendo
Copy link
Contributor Author

Any luck? I tried clearing cookies and creating an action to clear state. Couldn't get the cookie to clear though.

@mxstbr
Copy link
Contributor

mxstbr commented Aug 30, 2017

Neither, got my new laptop today so I only tried for like half an hour though. Was just about to dig in again, I think it's due to a race condition in requests, will try and figure it out!

@mxstbr
Copy link
Contributor

mxstbr commented Aug 30, 2017

The only thing I'm missing from the homepage is a list of some of the biggest communities. The search is nice and all but it's kinda hard to know what to search for when you barely even know what Spectrum's about.

Would be nice to show some of the top communities below that? Or maybe some small text saying "Try searching for 'React', 'SpecFM', 'JavaScript'?" or something like that?

@mxstbr
Copy link
Contributor

mxstbr commented Aug 30, 2017

Added signed out fallbacks for all major internal-only pages. Now going to dig into the signing out thing since that's totally my bug.

Falls back to Login for most pages and Splash for the home ones.
@mxstbr
Copy link
Contributor

mxstbr commented Aug 30, 2017

I think I just fixed signing out and in!! (it was a single option change in our session middleware...)

Deploy incoming: https://spectrum-vkpdsuwkdr.now.sh/ Please thoroughly test signing in and out of this before shipping!

@mxstbr
Copy link
Contributor

mxstbr commented Aug 30, 2017

The logout stuff doesn't work yet if you log in again, f%%%%%%%, getting closer.

@mxstbr
Copy link
Contributor

mxstbr commented Aug 30, 2017

Fuck logging out let's just remove that button and be done with it ugh

@superbryntendo
Copy link
Contributor Author

cool. i have to run, but I'm good with that for now. Maybe just comment out the whole dropdown for users for the time being?

@brianlovin
Copy link
Contributor

Let's just comment out the logout button. Easy access to settings is still good.

@mxstbr mxstbr mentioned this pull request Aug 30, 2017
14 tasks
@mxstbr
Copy link
Contributor

mxstbr commented Aug 30, 2017

Done!

@brianlovin
Copy link
Contributor

Great. Is this good to ship then?

@mxstbr
Copy link
Contributor

mxstbr commented Aug 30, 2017

I guess? Give it another full test but I think we're good to go.

@brianlovin brianlovin merged commit eca0269 into master Aug 31, 2017
@brianlovin brianlovin deleted the splash-page-v2 branch August 31, 2017 04:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants