search

gbowne1 / RadioLogger

A Radio Logging application build with NodeJS and ExpressJS
GNU General Public License v3.0
6 stars 6 forks source link

[TODO] TODO (See TODO.md) #9

Closed gbowne1 closed 1 year ago

gbowne1 commented 1 year ago

Need to add a logger.

app.use(logger); as a middleware function to add logging to the NodeJS and ExpressJS sever and application. morgan is already installed on the serverso we will use that as a logger.

I also installed helmet. This is a middleware tool to help protect our server from some well-known web vulnerabilities by setting HTTP response headers appropriately. It comes with a collection of several middleware functions that set security headers returned from our Express application

gbowne1 commented 1 year ago

Not on this particular page of the project.

gbowne1 commented 1 year ago

I see the issue. I didnt use d-flex mx- my- etc properly.

gbowne1 commented 1 year ago

yeah, this was totally my fault. using the wrong Bootstrap 5 classes. Flex works good on this, although it would work with Grid too. Grid might be better for this but, all I had to do was adjust the positions and margins. My bad.

How should we go about adding /profile route? It's accessible by the Profile in the user menu. I wanna make this a modal.

I will push up what I have in a bit.

gbowne1 commented 1 year ago

Pushed up the changes I made. Dashboard looks better, though not perfect. Still some minor adjustment to be done.

gbowne1 commented 1 year ago

Pictures need a bit of sizing help and dashboard needs routing for the individual pages, like /vhflog, /scanlog, /mwlog, /hamlog, /qsldb

I would like to extract the navbar as its own component. I think thats about all the initial reusability we will need to start with, other than the sidenav.

Most of the form pages need fixing starting with the navbar so it looks the same as the dashboard navbar.

Navbar in there is still missing fa-bars on the far left to open the drawer.

gbowne1 commented 1 year ago

I am going to add some boilerplate login, resetpassword and register tests for mocha/cha this evening. Will need to add one for authentication/authorization.

Am still learning both so I added what I could.

gbowne1 commented 1 year ago

@jzunigarce I went and fixed password reset POST inside the server.js but now it requires a ./user

Not sure of what to do here.

gbowne1 commented 1 year ago

ahh.. the fix was just changing the User required route to ./models/user server restarted fine. Pushing fix.

gbowne1 commented 1 year ago

I couldnt get the tests I added to work, but they are there so we can hack them out so they work.

gbowne1 commented 1 year ago

At some point we should move to Ecmascript 2021 but by then we should probably install prop-types or typescript. that way we can use import syntax instead of constants, vars, etc.

gbowne1 commented 1 year ago

@jzunigarce I am going to add some UI / UX issues tonight.