Skip to content

Conversation

@harshalhonde21
Copy link
Contributor

Developer Checklist

  • Followed guidelines mentioned in the readme file.
  • Followed directory structure. (e.g. ProjectName/{USERNAME}/...yourfiles)
  • Starred ⭐ the Repo (Optional)

Summary

add a summary here

Screenshot

attach screenshots/gifs here

Live Project Link

add a working project link here

@PBJI
Copy link
Collaborator

PBJI commented Oct 14, 2023

@harshalhonde21 great, but I would like to add one thing here, Why don't we just collaborate on your cloned repo, pulling requests there and then mergin the complete front-end here, this will keep the process cleaner, as the sole purpose of pull requests in this repo is to add new projects mostly. I'll keep a watch on your clone and we'll move it from there onwards.

@NitkarshChourasia ?

@harshalhonde21
Copy link
Contributor Author

Ya, it is cleaner process I absoulty agree from u but @PBJI when we work the project combination here the audience will go with us to collabrate to the main project please I am request you @PBJI to continue the project here only and please give only 5 days i will promise to complete the work on it hole frontend here

@harshalhonde21
Copy link
Contributor Author

please merge the readme PR to see the all users

@NitkarshChourasia
Copy link
Collaborator

NitkarshChourasia commented Oct 14, 2023 via email

@harshalhonde21
Copy link
Contributor Author

No @NitkarshChourasia it will make more difficulty to do that because it is good pratice to do PR on main repo. Okk for now please merge the readme.md at least because it is neccessary to know all the users.

@PBJI
Copy link
Collaborator

PBJI commented Oct 14, 2023

@harshalhonde21 okay, I'll merge it also I will request you to update the main repo's readme as well and at the header notify users about this prpject that we are pursuing which I hope get them willing to contribute as well.

@PBJI PBJI merged commit c37de76 into thinkswell:master Oct 14, 2023
@harshalhonde21
Copy link
Contributor Author

yes, okk thanks to understand me my request the work ur given I will do it in 15 mins and the pr will be seen to you

@harshalhonde21
Copy link
Contributor Author

I have changed the main readme of the repo please see and review and merge it.

@NitkarshChourasia
Copy link
Collaborator

@harshalhonde21 I see your point.

@NitkarshChourasia
Copy link
Collaborator

@harshalhonde21 @PBJI whatever is to the bes interest of the team as whole.

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.

3 participants