hazelcast / hazelcast-platform-operator-docs

Source content for the Kubernetes Operator of Hazelcast Platform (Enterprise)
3 stars 13 forks source link

[CN-499] Add Documentation for Restore from HotBackup CR #59

Closed mtyazici closed 1 year ago

netlify[bot] commented 1 year ago

Deploy Preview for pedantic-goldberg-f76ec1 ready!

Name Link
Latest commit d709f0fa6440d0206ae58e1b7e141b5cb945d246
Latest deploy log https://app.netlify.com/sites/pedantic-goldberg-f76ec1/deploys/636a3db08850b900090cbec7
Deploy Preview https://deploy-preview-59--pedantic-goldberg-f76ec1.netlify.app/operator/latest-snapshot/backup-restore
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

angelasimms commented 1 year ago

Mainly minor edits but just a couple of points around consistency:

  • I've checked and we use Hazelcast Platform Operator rather than operator etc.
  • Where you have an instruction next to a code snippet, if the instruction is a full sentence then you don't need a colon.

Examples:

Apply a resource.

Restore a resource.

The only time you need to use a colon is for a incomplete sentence.

For example:

Use the following command:

@mtyazici. I think you had a couple of queries on comments but they look like they have all been resolved. Do you need me to review again?

mtyazici commented 1 year ago

@angelasimms, some of the suggestions were in two different comments so I thought there was a problem but it was OK. It would be great if you could review it one final time.