-
Notifications
You must be signed in to change notification settings - Fork 158
Update Feature/issue 608 #624
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
base: main
Are you sure you want to change the base?
Conversation
|
So normally it's better to just add the changes to the currently open PR, could you do that? feel free to ask questions if you're not sure how to. you can just make changes and then push to that same branch and it will show in the same PR |
Okay, yeah, I didn’t really understand how to add new changes. |
merrua
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have a file called "cd" that looks like it should not be in your pull request.
You have a file called git that shouldn't be checked in.
Take a look at the .gitignore file that prevents unwanted files from being checked in.
This reverts commit 73f22e7. �
Okay, I’ve just uncommitted them, no idea how they got in |
|
looks good, can you squash this down to one commit then we're good to merge |
|
@snikmas Did you see Devins comment about squashing the commits? |
now I see it. I will try to squash it |
|
@snikmas checking in on this |
Description
Wiki moved to [this main/docs]](https://github.com/Together-100Devs/Together/tree/main/docs), so I updated the url
Type of change
Please select everything applicable. Please, do not delete any lines.
Issue
fixed header buttons on /calenadr #608 (review)
Checklist:
npm run testandnpm run test:e2eand all tests have passed successfully or I have included details within my PR on the failure.npm run lintand resolved any outstanding errors. Most issues can be solved by executingnpm run format