Neondb branching

[Myestery]

/claim #70

[height]

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

[dray92]

@Myestery can you check the lint errors?

[Myestery]

@Myestery can you check the lint errors?

Okay

[Myestery]

All good @dray92

[Sid-Lais]

1. We could use an uncropped screenshot as we do in our other docs for the architecture.
2. Used a different format to link images and videos.

Used here is:

<figure><img src="../../.gitbook/assets/neondb.drawio.png" alt=""><figcaption></figcaption></figure>

We used:

![Azure VM Architecture](../../../.gitbook/assets/azure-vm-architecture.png)

[Myestery]

1. We could use an uncropped screenshot as we do in our other docs for the architecture.
2. Used a different format to link images and videos.

Used here is:

<figure><img src="../../.gitbook/assets/neondb.drawio.png" alt=""><figcaption></figcaption></figure>

We used:

![Azure VM Architecture](../../../.gitbook/assets/azure-vm-architecture.png)

I was able to change no 2 but for no 1, I'll like to know which better tool I could use to achieve it cos I used draw io and it had to be cropped

[Sid-Lais]

I was talking about the Create a new branch screenshot. For diagrams we use Lucid app

[Myestery]

I was talking about the Create a new branch screenshot. For diagrams we use Lucid app

Fixed now with a better screenshot.

[Sid-Lais]

In line 96 I guess launch command should be npm start which you mentioned earlier
The video just goes through the things and does not explain them clearly. Most importantly, the video voice is very low. You could try the app in the end by running some commands or something to show it works.
Should install postgres cli to check the postgres connectivity

[Sid-Lais]

I was talking about the Create a new branch screenshot. For diagrams we use Lucid app

Fixed now with a better screenshot.

Delete the prev one

[dray92]

@Myestery can you take a look?
rest of the PR seems good and we're ready to merge after this

[Myestery]

Yeah I'll get it done in 24 hours

[Myestery]

@Myestery can you take a look? rest of the PR seems good and we're ready to merge after this

Hello @dray92, I tried doing the tutorial again to replace the existing video but I can't load the DATABSE_URL env variable like I did before.
It looks like something has changed about the platform that I'm not aware of from the docs

[Myestery]

Ready for merge

Hello @dray92 @Sid-Lais
I was contacted by someone from the team and I was able to redo the video using team secrets
It should be ready for merge now

[Sid-Lais]

@Myestery can you check above as I have had put some comments that needs to be resolved

[Myestery]

@Myestery can you check above as I have had put some comments that needs to be resolved

I have resolved those ones too

[Sid-Lais]

@Myestery can you check above as I have had put some comments that needs to be resolved

I have resolved those ones too

I meant the minor grammatical errors

[Myestery]

@Myestery can you check above as I have had put some comments that needs to be resolved

I have resolved those ones too

I meant the minor grammatical errors

I have wor

@Myestery can you check above as I have had put some comments that needs to be resolved

I have resolved those ones too

I meant the minor grammatical errors

I have worked on all the reviews you gave that I can find here. If there are any other ones, please point me to them

[Sid-Lais]

in how-to-guides/third-party/neon-db.md

1. line 19 It will be Current point in time not to time and make it bold
2. line 124 in generating not ingenerating

[Myestery]

done!

in how-to-guides/third-party/neon-db.md

1. line 19 It will be Current point in time not to time and make it bold
2. line 124 in generating not ingenerating

done!