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

Added fixes and installation documentation #68

Merged
merged 1 commit into from
Jan 12, 2020
Merged

Added fixes and installation documentation #68

merged 1 commit into from
Jan 12, 2020

Conversation

Sid911
Copy link
Contributor

@Sid911 Sid911 commented Jan 10, 2020

  • Please check if the PR fulfills these requirements
  • The commit message follows our guidelines
  • Docs have been added / updated (for bug fixes / features)
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
  • Bug fix
  • Feature implementation
  • Doc updates
  • What changes have you introduced?
  • Preview / Steps to verify your work:
    image

@thewasif
Copy link
Member

This is for developers. Isn't there a way to install production build? 🤔🤔

@Sid911
Copy link
Contributor Author

Sid911 commented Jan 10, 2020

yeah there is... its in install branch but that never gets builds because build system isn't working atm

@Sid911
Copy link
Contributor Author

Sid911 commented Jan 10, 2020

@fcartegnie @hongquan @mariobehling can this be reviewed 😇 thanks

desktop.md Outdated Show resolved Hide resolved
@Sid911
Copy link
Contributor Author

Sid911 commented Jan 11, 2020

@fcartegnie Done :)

desktop.md Outdated Show resolved Hide resolved
desktop.md Outdated Show resolved Hide resolved
@Sid911
Copy link
Contributor Author

Sid911 commented Jan 11, 2020

@fcartegnie fixed typo and added formatting, if that meant this
image 😅

@Sid911
Copy link
Contributor Author

Sid911 commented Jan 11, 2020

@cweitat please review and merge 😀

@cweitat cweitat self-requested a review January 12, 2020 05:55
desktop.md Outdated Show resolved Hide resolved
desktop.md Outdated Show resolved Hide resolved
@Sid911 Sid911 requested a review from cweitat January 12, 2020 08:53
desktop.md Outdated Show resolved Hide resolved
desktop.md Outdated Show resolved Hide resolved
@Sid911 Sid911 requested a review from cweitat January 12, 2020 11:32
@Sid911
Copy link
Contributor Author

Sid911 commented Jan 12, 2020

@cweitat I think that's it.. for typos that I could spot 😅.. I just copy pasted that to word 😂

desktop.md Show resolved Hide resolved
@Sid911 Sid911 requested a review from cweitat January 12, 2020 13:27
@Sid911
Copy link
Contributor Author

Sid911 commented Jan 12, 2020

@cweitat that section is already made.. just the link wasn't correct as it was changed from .rst to .md,I changed just that part :)

I guess you didn't expand and just saw the changes section 😅 and didn't see the preview

image

Copy link
Contributor

@cweitat cweitat left a comment

Choose a reason for hiding this comment

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

yup, my bad

@cweitat cweitat merged commit 25bf4ca into fossasia:master Jan 12, 2020
kumuditha-udayanga pushed a commit to kumuditha-udayanga/pslab-documentation that referenced this pull request Jan 15, 2020
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.

Insufficient Doc for Desktop version of Pslab
5 participants