Scholar Userguide Migration#236
Conversation
|
Can you add your name to the md pages? This way we can let you know when there are edits needed in the future. |
| - Scholar | ||
| authors: | ||
| - jin456 | ||
| resource: Scholar |
There was a problem hiding this comment.
For naming consistency, can we have different variable to define Scholar since now we are refering to lower case variable as resource?
There was a problem hiding this comment.
@Guangzhen-Jin I can make a new variable "cluster: Scholar" for the upper-case version. Is that variable name okay?
There was a problem hiding this comment.
Cool, I made this change in this commit
There was a problem hiding this comment.
It's better to have h1 header on every page for accessibility.
There was a problem hiding this comment.
Added missing h1 headers in this commit
There was a problem hiding this comment.
You might want to use the macro snippet under main.py.
There was a problem hiding this comment.
You might want to use the macro snippet under main.py.
There was a problem hiding this comment.
You might want to use the macro snippet under main.py.
There was a problem hiding this comment.
You might want to use the macro snippet under main.py.
There was a problem hiding this comment.
You might want to use the macro snippet under main.py.
There was a problem hiding this comment.
You might want to use the macro snippet under main.py.
There was a problem hiding this comment.
I don't see a macro snippet for this one, leaving as-is for now
There was a problem hiding this comment.
You might want to use the macro snippet under main.py.
There was a problem hiding this comment.
You might want to use the macro snippet under main.py.
There was a problem hiding this comment.
You might want to use the macro snippet under main.py.
There was a problem hiding this comment.
You might want to use the macro snippet under main.py.
There was a problem hiding this comment.
You might want to use the macro snippet under main.py.
Guangzhen-Jin
left a comment
Guangzhen-Jin
left a comment
There was a problem hiding this comment.
Looking good now.
2 participants
This PR includes all migrated content for the Scholar userguide.