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

General error layout #26

Merged
merged 11 commits into from
Oct 4, 2021

Conversation

shivamparihar12
Copy link
Contributor

So here is the another pr. since there are no conflicts and .idea files are also deleted then i guess this one must be fine. if not so then please tell what should i do now.

@justdvnsh
Copy link
Owner

@shivamparihar12 . I am accepting this PR, and i will delete the .idea files myself. But for further Pr's please make sure to update your repo with develop branch. It has the .gitignore file which will automatically ignore the .git files from the commit.

@justdvnsh
Copy link
Owner

Also, before i merge the pr, can you share a screenshot of the layout ?

@justdvnsh justdvnsh added ready-to-merge attached when a pr is ready to be merged UI-change Label to put if there is any UI change labels Oct 4, 2021
@shivamparihar12
Copy link
Contributor Author

Thanks and sorry for the trouble.
here is the screenshot of the layout.
Screenshot (697)

@justdvnsh
Copy link
Owner

justdvnsh commented Oct 4, 2021

@shivamparihar12 No worries. I can handle that. Also, please star the repo, and thanks for your contribution

@justdvnsh justdvnsh added the hacktoberfest-accepted Attached when pr is ready to merge and can be counted for hacktoberfest. label Oct 4, 2021
@justdvnsh justdvnsh merged commit 5f7d9b2 into justdvnsh:develop Oct 4, 2021
@shivamparihar12 shivamparihar12 deleted the general-error-layout branch October 26, 2021 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Attached when pr is ready to merge and can be counted for hacktoberfest. ready-to-merge attached when a pr is ready to be merged UI-change Label to put if there is any UI change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants