-
Notifications
You must be signed in to change notification settings - Fork 43
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
Basic start of Broker Transition Solution Documentation #1570
Conversation
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.
Plenty of good stuff here, but I have some comments. Complete those and we can have a new review before a translation to Norwegian.
content/broker/broker-transition/how-to-get-started/_index.en.md
Outdated
Show resolved
Hide resolved
content/broker/broker-transition/how-to-get-started/_index.en.md
Outdated
Show resolved
Hide resolved
Merge remote-tracking branch 'upstream/master'
Update of doc based on prcomments
updated failing link
fix link attempt 2
content/broker/broker-transition/technical-overview/_index.en.md
Outdated
Show resolved
Hide resolved
content/broker/broker-transition/technical-overview/_index.en.md
Outdated
Show resolved
Hide resolved
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.
Looks good, just need the translation to Norwegian.
lagt inn oversatte sider
Norwegian translated pages have now been added. |
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.
In general looks OK, but needs consistent use of translated terms as this differs some places.
Some terms like status codes/operations should not be translated to Norwegian, but instead quoted in original English versions,
Some of the translated text needs to be cleaned up/refactored slightly due to whitespacing and/or grammatical inconsistencies in norwegian versions..
updated based on comments
fmissed one
oppdatert overgangsløsning
…it les norweglish
Translate comments4
should resolve the last converastions
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.
Looks good :)
Basic start of Broker Transition Solution Documentation
Description
Basic start of Broker Transition Solution Documentation