Thanks, the patch applied.
On Wed, Jun 15, 2022 at 9:52 AM Khushboo Vashi <
khushboo.va...@enterprisedb.com> wrote:
>
>
> On Tue, Jun 14, 2022 at 6:26 PM Yogesh Mahajan <
> yogesh.maha...@enterprisedb.com> wrote:
>
>> Hello,
>>
>> Please find the updated patch which validates the field after the
On Tue, Jun 14, 2022 at 6:26 PM Yogesh Mahajan <
yogesh.maha...@enterprisedb.com> wrote:
> Hello,
>
> Please find the updated patch which validates the field after the blur
> event on cluster name.
>
>
Looks good to me.
Thanks.
Thanks,
> Yogesh Mahajan
> EnterpriseDB
>
>
> On Tue, Jun 14, 2022 at
On Tue, 14 Jun 2022, 13:29 Yogesh Mahajan,
wrote:
> Hi Khushboo,
>
> Thanks for reviewing the patch.
>
> On Tue, Jun 14, 2022 at 11:35 AM Khushboo Vashi <
> khushboo.va...@enterprisedb.com> wrote:
>
>> Hi Yogesh,
>>
>>- Spelling mistake in below error message
>> - Name must be more than
Hi Yogesh,
- Spelling mistake in below error message
- Name must be more than 3 characters or more & *Shoudld* not have
capital letter
- Cluster name validation comes after all the fields are validated. It
should be done while giving input for the cluster name
- Even If the
Hi Yogesh,
Review comments:
- Step 2: The below statements should be in a different line, like
-
- "Azure CLI" will use the currently logged in identity through Azure
CLI on the local machine.
- "Interactive Browser" opens a browser to authenticate a user
interactive
On Wed, Jun 1, 2022 at 3:28 PM Akshay Joshi
wrote:
> Hi Khushboo
>
> Can you please review it?
>
> Will do it tomorrow.
> On Wed, Jun 1, 2022 at 10:11 AM Yogesh Mahajan <
> yogesh.maha...@enterprisedb.com> wrote:
>
>> Hi,
>>
>> Please find the attached patch which provides functionality to deplo
Hi Khushboo
Can you please review it?
On Wed, Jun 1, 2022 at 10:11 AM Yogesh Mahajan <
yogesh.maha...@enterprisedb.com> wrote:
> Hi,
>
> Please find the attached patch which provides functionality to deploy a
> Postgres cloud instance on Azure Postgresql.
>
> Thanks,
> Yogesh Mahajan
> Enterpris
