Maintainers, Collaborator and Contributors

[gbowne1]

I would like to propose that this open source project have several maintainers, reviewers, collaborators and contributors. My goal here is a minimum of one person per role, two with some coverage.

Frontend

Write and maintain the frontend code.

Implement new features

Debugging & testing browser and device specific bugs

Backend/Server

Maintain the code for the nodejs and expressjs server including the index.js / server.js

Database

Maintain the mongodb code, schemas, models, collection(s)

Testing QA & QC

Writing and maintaining the e2e, unit and integration test code

Still need to decide which testing platforms we will use. Jest and React Testing Library are built in. We can start there.

• Accessibility testing

• Cross Browser Testing

• Device testing

Documentation

Write, review and maintain application including international documentation.

Deployment & DevOps

Make sure that builds run, containerized builds, etc complete

Deployment to server

At this point in time I have been requiring a minimum of 5 successful merges before someone can be invited to be a collaborator and maintainer of the project.

@LOGESH-B

[gbowne1]

I also think that at certain point we should require a minimum of 2 reviews on PRs from project maintainers in order to merge. Certain types of frontend and backend, etc can be assigned via a CODEOWNER file in the project root folder/ directory.

[LOGESH-B]

Hii @gbowne1
I am new to Open source contribution,I have done couple of projects Node,React,MongoDB and I don't have any exposer in the areas of testing and devops..So you can assign accordingly🧑‍💻✌️

[gbowne1]

@LOGESH-B

We can definitely assign testing and devops people as maintainers.

Theres still a few who have 3 and 4 succesfully merged PRs already.

The larger the project gets the easier it will be to maintain with at least 1 or 2 people per role and it would get documented.

The contributions we have had so far have been pretty solid even if a few were pretty small.

[gbowne1]

@Vishwanath760970 @shivm29 @neeravbhaskarla @k-deepak04 @kabszac @Mayor-Rhymes @Yuva0 @Piyush-Deshmukh @kevinstewartsmith @javadkhangurbanov @goutham03062001 @atharvarekhawar @akansjain @777Prabhjot @giri-madhan @khalil-p @realrohitgurav @shyamtawli @victor20m

[k-deepak04]

@gbowne1 i can manage the fronend and backend task, but for now quite busy in open-source program, but will look the pr's issues etc once a week and help with them :)

[gbowne1]

You already have contributed a little. 3 or 4 times I think. That's great. Thanks! :-) Once you reach 5 successful merged PRs I will invite you to be a collaborator to the project. Feel free to contribute when you are able to.

[gbowne1]

We already have one person helping with the backend. I think at least once a week is a solid target to work on something.

[k-deepak04]

okay i will, i guess i have contributed more than 5 but it's in your other repos also, i will contribute in this one too :)

[k-deepak04]

i will look into frontend and functionality aspect mainly.

[gbowne1]

Yes a few more than 5 total. I chose 5 successful merges in order to be invited to be collaborator as we have gotten a lot done and 5 gives us a chance to see how solid your contributions are.

The more people we have contributing and maintaining the app, the easier it will be.

[shyamtawli]

Hey I can look into frontend part.

[gbowne1]

Ok. @shyamtawli Plenty of Frontend stuff to do

[Vishwanath760970]

@gbowne1 I hope you are doing well, I see the backend implementation is started, good to see that, I've been busy lately with some interviews and office work, but I'll start looking into both frontend/backend tasks shortly.

[gbowne1]

Hi @Vishwanath760970

All is well.

Yes, the backend has been started. Still a ways to go.

Feel free to work on anything you like as you are you are able to.

Thanks :-)

[gbowne1]

@BlackBond06 @alienqqe @atharvarekhawar @jchansen925 @victor20m @kabszac @javadkhangurbanov @giri-madhan @Jonnie-Dev @khalil-p @shyamtawli @Piyush-Deshmukh @ratishjain12 @neeravbhaskarla @realrohitgurav @shivm29 @shyamtawli @Yuva0

[BlackBond06]

@gbowne1, when I get to 5 successfully merged PRs, I would like to be part of maintaining the frontend part of the app.

[gbowne1]

Great. This project has come a long ways. We are almost to the point we will need QA/QC and to help @LOGESH-B and I review PRs to reduce the amount of errors.

Eventually I will require passing tests to be included with PRs. Something that should be done as soon as possible.

[BlackBond06]

The project is really coming along very well I must say. I particularly love the enthusiasm shown by the contributors. I'm not really familiar with unit testing. Maybe I can make a research and see if I can help in that regard also. If you have resources on unit testing please share.

[gbowne1]

https://jestjs.io/docs/getting-started is the best place to start.

App.test.js is included and this file comes out of the box when you run create react app and I think it's the most under used feature. This file was meant to test App.js

As an example Login testing would be most likely in Login.test.js and would be coded to test login functions.

These files are normally kept in a /test or /tests or /__test, /__tests or even an _ _ tests _ _ folder. I prefer /tests but I would follow what the Jest docs say.

[gbowne1]

There is a couple videos I would suggest for learning Jest. I would start here, It is an hour long but provides a solid overview.

https://youtu.be/7r4xVDI2vho?si=lSwQKorVNRlqBhfT

[BlackBond06]

I definitely will check this out. Thank you

[gbowne1]

You're welcome. Glad to be of help. 😀

[LOGESH-B]

I am also started learning to writing test using jest✌️

[BlackBond06]

Thank you.

[gbowne1]

You're welcome :-)