-
Notifications
You must be signed in to change notification settings - Fork 14
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
Add link to training materials #831
Conversation
PCT essentials
This is the standard value and we had https which is invalid
Style tag is not valid in the body tag, also this CSS is not used.
525ba8c
to
3f96fcb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code is fine but might want others to check if they are happy with content. 👍
HTML tidy
Also this PR should be to master now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why have you got a PR to merge into https://github.com/npct/pct-shiny/tree/pct_essentials branch? Maybe I've missed something but this is a bit confusing.
I don't anymore. I created it before the essentials branch was merged and left it open deliberately to make the useful conversations there more prominent. I should have said 'Replaces #831 ' in the discussion of #835 . Will do the that with this one. |
No description provided.