On Tue, 14 Jun 2022, 13:29 Yogesh Mahajan, <yogesh.maha...@enterprisedb.com> 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 3 characters or more & *Shoudld* not have >> capital letter >> >> Done. > >> >> - Cluster name validation comes after all the fields are validated. >> It should be done while giving input for the cluster name >> >> Current react framework validates parent schema fields only after all > children schemas. Hence cluster name is validated after all children schema > fields. > Then, I think we should fix this. > >> - Even If the credentials are incorrect, the authenticate button gets >> disabled. So, I can't update and re-authenticate >> >> Done. > >> Apart from this, it looks good to me. >> >> Thanks, >> Khushboo >> > > Thanks, > Yogesh Mahajan > EnterpriseDB > > >> >> On Fri, Jun 10, 2022 at 8:14 PM Yogesh Mahajan < >> yogesh.maha...@enterprisedb.com> wrote: >> >>> >>> Hi, >>> >>> Please find the updated patch with documentation. >>> >>> >>> On Mon, Jun 6, 2022 at 1:10 PM Khushboo Vashi < >>> khushboo.va...@enterprisedb.com> wrote: >>> >>>> 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 >>>> interactively. >>>> >>>> Done. >>> >>>> >>>> - Disable the next button once authentication is complete. >>>> >>>> As discussed disabled Authentication button once authentication is >>> completed. >>> >> >>>> - Cluster name availability call calls the server on every field >>>> change >>>> >>>> Done. >>> >>>> >>>> - Availability zone needs description >>>> >>>> This is a generic term with cloud hence not added. >>> >>>> >>>> - Add High availability option >>>> >>>> Done >>> >>>> >>>> >>>> Code: >>>> >>>> - check_cluster_name_availability should be using the GET method >>>> instead of Post >>>> >>>> Done. >>> >>>> >>>> - Fix SonarLint issues >>>> >>>> Done. >>> >>>> >>>> - Do we need the cache_persistence_options as it will create the >>>> persistent storage which we do not require I guess? >>>> >>>> Yes, it is required. >>> >>>> >>>> - Why do we need to call _get_azure_credentials on every request? >>>> Can't we store it in the session object? >>>> >>>> Function returns without calling again credentials if an existing >>> client is present. >>> >>>> >>>> - Use gettext wherever required in the js file >>>> >>>> Done. >>> >>>> >>>> Thanks, >>>> Khushboo >>>> >>> >>> Thanks, >>> Yogesh Mahajan >>> EnterpriseDB >>> >>> >>>> >>>> On Wed, 1 Jun 2022, 10:11 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 >>>>> EnterpriseDB >>>>> >>>>