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

PULL REQUEST #65

Open
Fatmasiam opened this issue Oct 29, 2019 · 1 comment
Open

PULL REQUEST #65

Fatmasiam opened this issue Oct 29, 2019 · 1 comment
Assignees

Comments

@Fatmasiam
Copy link
Collaborator

Fatmasiam commented Oct 29, 2019

When you open the PR :

  1. add it to the related project.
  2. remove any console.log except the console.log on the error file.
  3. remove unrelated files
  4. make sure that your branch is up to date with master
  5. make sure that is working locally
  6. make sure that there are no ESlint issues in your code
  7. code is consistent with an agreed team style
  8. write a description for what the component or page do
  9. write how to test your component

-- put in progress label when you are still working on the PR.
or make it draft

AND KEEP IN YOUR MIND THAT THE ISSUES IT IS REFERENCE

  • Its didn't take our time to neglect it
@Fatmasiam
Copy link
Collaborator Author

@yosefalnajjarofficial @alaa-yasin @AlaaTaima @fares98

follow this issue on the PR

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

No branches or pull requests

5 participants