Skip to content

Docs: Documented the Workflow #2

Merged
17 commits merged intomainfrom
jeel-dev
Aug 12, 2024
Merged

Docs: Documented the Workflow #2
17 commits merged intomainfrom
jeel-dev

Conversation

@JeelDobariya38
Copy link
Member

@JeelDobariya38 JeelDobariya38 commented Aug 5, 2024

TL;DR: Add Documentation

Changes Made

  • add description in README.md
  • document the release process and database design
  • all other documents.

@JeelDobariya38 JeelDobariya38 requested review from a user and kudanilll August 5, 2024 14:47
@JeelDobariya38 JeelDobariya38 added the enhancement New feature or request label Aug 5, 2024
@kudanilll kudanilll marked this pull request as ready for review August 5, 2024 16:24
@JeelDobariya38 JeelDobariya38 changed the title Feat: Made A Basic GUI Application (non-functional). Docs: Aug 5, 2024
@JeelDobariya38 JeelDobariya38 changed the title Docs: Docs: Add Documentation Aug 5, 2024
Copy link
Member

@kudanilll kudanilll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm, the commit is 'add docs', but files changes is not docs.

@JeelDobariya38
Copy link
Member Author

Wait I am doing!!

@JeelDobariya38
Copy link
Member Author

It is under development not ready to review

@JeelDobariya38 JeelDobariya38 added documentation Improvements or additions to documentation and removed enhancement New feature or request labels Aug 5, 2024
@JeelDobariya38 JeelDobariya38 changed the title Docs: Add Documentation Docs: Documented the Workflow Aug 5, 2024
JeelDobariya38 and others added 3 commits August 5, 2024 23:17
change 'Password-Manager' to 'Password Manager'
Copy link
Member

@kudanilll kudanilll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I changed some files to make it more organized (README.md & database-design.md)

@JeelDobariya38 JeelDobariya38 marked this pull request as draft August 6, 2024 10:44
@JeelDobariya38 JeelDobariya38 marked this pull request as ready for review August 6, 2024 13:23
@JeelDobariya38
Copy link
Member Author

JeelDobariya38 commented Aug 6, 2024

Please add your personal email in the Security policy @achmaddaniel24 @HamadaNative

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are you deleting the .idea is that for C.A

@JeelDobariya38
Copy link
Member Author

JeelDobariya38 commented Aug 6, 2024

Yes, it was done to avoid conflict.

@JeelDobariya38 JeelDobariya38 requested a review from a user August 6, 2024 13:47
@kudanilll
Copy link
Member

Please add your personal email in the Security policy @achmaddaniel24 @HamadaNative

okay

Copy link
Member

@kudanilll kudanilll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good documentation design!

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok. I don't see anything wrong.

@ghost ghost merged commit e53bcd3 into main Aug 12, 2024
@JeelDobariya38 JeelDobariya38 deleted the jeel-dev branch August 18, 2024 14:17
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants