Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Front End Modification #127

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

hammadsaleemm
Copy link

I have changed the title and did front end changes on the viewer along with that ive worked on the backend modules to change the redundant ones and installed the new ones.

@rrrokhtar
Copy link

Actually it seems a lot of changes on your pull request, I will recommend you to briefly describe what are the changes and separate each feature or modification in a single pull request to be easy to review and to get merged.
I will recommend you to read this blog entry to get some insights about what is a perfect pull request should be.
https://betterprogramming.pub/how-to-make-a-perfect-pull-request-3578fb4c112

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants