search

magnifydev / magnify

A Digital Program of Studies for LMHS.
https://magnify.web.app
Apache License 2.0
8 stars 0 forks source link

Add info tab to navigation bar #20

Open Ascent817 opened 2 years ago

Ascent817 commented 2 years ago

This should outline the aims of this project, how to contact the developers, maybe even a changelog.

GoogolGenius commented 2 years ago

I have ongoing changes that will add a contact form and that will be PRed tomorrow when I can. Can the changelog be on GitHub? How do you plan for that. Personally I think the link to the GitHub repository with a decently updated README will be fine to outline the purpose of Magnify.

GoogolGenius commented 2 years ago

~Also I'd like your feedback on the contact form, currently I'm using another web service to handle the requests and such, however I'm thinking we might as well use PHP to handle the form instead.~

Nevermind this requires an SFTP server or whatever it's called I think. The current impl is fine.

Ascent817 commented 2 years ago

I don't know about using a readme. As soon as it links to a 'code site', a lot of ppl will think it's beyond them and close it. I know from before I knew how to code, when things linked to GitHub I immediately assumed they were asking for code help and clicked off. It would be better if there was a modal displaying that information on-site.

GoogolGenius commented 2 years ago

I think a modal pop up could be fine. Unless we want to render an entirely new page but as a single page React app still? I don't think we really need routes.

Ascent817 commented 2 years ago

We don't need routes. I think a modal is fine.

GoogolGenius commented 7 months ago

Add last updated and endorsement from Linn-Mar at some point