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

Update 07-vnc-server.md #1486

Closed
wants to merge 3 commits into from
Closed

Update 07-vnc-server.md #1486

wants to merge 3 commits into from

Conversation

Amiga500
Copy link
Contributor

@Amiga500 Amiga500 commented Mar 7, 2024

No description provided.

@XK9274
Copy link
Member

XK9274 commented Apr 8, 2024

Hey @Amiga500 - thanks for all of your contributions and additions to the docs, it's hugely appreciated by all of us here.

Would it be possible for you to merge all of your docs PR's into one single PR please? I know it's easier on web to do a PR for each but it causes us overhead (and spams our trackers)

If you need a hand with anything give me a shout, or if you're unable to, that's fine - we'll do it this time.

@Amiga500
Copy link
Contributor Author

Amiga500 commented Apr 9, 2024

Hey @XK9274, sure, i have only to see how to do it! Thanks to you

@Amiga500
Copy link
Contributor Author

Amiga500 commented Apr 9, 2024

PR created and made some clean

@XK9274
Copy link
Member

XK9274 commented Apr 9, 2024

PR created and made some clean

Thanks! Will take a look after work

@XK9274 XK9274 changed the title Update 07-vnc-server.md Docs fixes and amendments Apr 9, 2024
@XK9274 XK9274 changed the title Docs fixes and amendments Update 07-vnc-server.md Apr 9, 2024

1. **Enable** - Activate VNC Server
2. **Framerate** - Set the framerate of the VNC Server between 1 and 20. The higher the framerate, the more CPU will be use.
2. **Framerate** - Set the framerate of the VNC Server between 1 and 20. The higher the framerate, the more CPU will be use
Copy link
Member

Choose a reason for hiding this comment

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

You ok to add the "d" on the end of "use" (used) then this one's good to go - Thanks!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added on main PR. I close this one. Thanks!

@XK9274
Copy link
Member

XK9274 commented Apr 9, 2024

I've requested changes here but, if the changes in this PR are also included in your new one, feel free to close this one off :)

@Amiga500 Amiga500 closed this Apr 9, 2024
@Amiga500 Amiga500 deleted the patch-32 branch May 4, 2024 04:20
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.

2 participants