that-website icon indicating copy to clipboard operation
that-website copied to clipboard

dead submit button on milestone section

Open theClarkSell opened this issue 5 years ago • 6 comments

the home page has a dead link. It could be coming from the api, or not even wired up.

image

theClarkSell avatar Jan 22 '20 16:01 theClarkSell

This is more complicated than I assumed on the surface.
This link button is controlled from the database in the notifications sub-collection of events.

The button uses next/link object and we are sending it an entire URL instead of the page/path. The page/path ('wi/call-for-counselors') cannot be set as GraphQL is set to have that value as a URL. To set the path only, breaks this condition.

The database has been set with the href of: https://www.thatconference.com/wi/call-for-counselors

The button still doesn't direct the user to the Call for Counselors page. I believe this is due to the fact that it is the same domain and should be a page reference, not a full URL reference.

I think the correct fix here is in GraphQL to store the path (string) not a URL. Looking for a second on this:

@csell5 @kburnell @saragibby

brettski avatar Jan 22 '20 17:01 brettski

Based on this thread: https://github.com/zeit/next.js/issues/8555 The suggestion is to replace the Link component with an as Link will be removed later. Currently all pages are SSR so using Link really has no benefit when it comes to refeshing the page. Using Link or will both cause the page to refresh as /call-for-counselors is a server side page.

brettski avatar Jan 22 '20 21:01 brettski

@brettski I'm confused with this - when I click on the button, I am taken to the www.thatconference.com/wi/call-for-counselors page. So this really isn't a 'dead' button, or has something changed since your last comment?

mcookWI avatar Jan 27 '20 14:01 mcookWI

@mcookWI There is a hack in place currently so it works. This hack changes the referenced link from: https://www.thatconference.com/wi/call-for-counselors to: http://www.thatconference.com/wi/call-for-counselors

The assumption on the hack is that Link sees the the link as external and redirects to it. If I put it back to the From you can reproduce the issue.

The correction here is our approach to managing the link. We can't make it path as I suggested earlier as this is a notification link which could be used in any medium, not only the web site.

I am pretty sure Sara had a suggestion for it.

brettski avatar Jan 27 '20 15:01 brettski

@saragibby Do you recall the suggestion for this?

mcookWI avatar Feb 07 '20 19:02 mcookWI

Yeah, I don't recall anything about this one... but let's pause for a moment... February 7th, 2019... we were with people, sitting in coffee shops, complaining about loud restaurants... ahhh, those were the days.

So seeing as how this was essentially 3 years ago, what do we need to do for this button in the important dates section?

saragibby avatar Nov 05 '20 00:11 saragibby