search

Azure / Commercial-Marketplace-SaaS-Accelerator

A reference example with sample code for developers interested publishing transactable, Software as a-Service offers in the Microsoft commercial marketplace.
MIT License
179 stars 276 forks source link

WebNotificationUrl Admin Setting Missing #610

Closed adrian-spear closed 7 months ago

adrian-spear commented 7 months ago

WebNotificationUrl admin setting is missing from the app settings config in Admin app. As per this video: https://www.youtube.com/watch?v=A7TKSE08dAY&t=277s at 04:37.00 - I am attempting to set up external notification feature. I installed Accelerator from Azure Marketplace but have published from the latest main branch to no avail.

To Reproduce Steps to reproduce the behavior:

  1. Go to Admin Settings
  2. WebNotificationUrl setting is missing

Expected behavior Expected 'WebNotificationUrl' setting to be available.

Screenshots None.

Environment (please complete the following information): Deployed to App Service (Windows) as per instruction provided in your video how to install from Azure Marketplace.

santhoshb-msft commented 7 months ago

@adrian-spear which version are you using? latest 7.4.0? in that case then should see the setting coming in from the database,

adrian-spear commented 7 months ago

Hi,

The latest migration in the cloned source code is ‘20230912052848_SubscriptionDetails_v740’ so I’m going to say ‘Yes’ to 7.4.0. The code for both admin and customer portals has been deployed using Visual Studio Publish.

The migrations history in the database is still showing ‘20221118211554_Baseline_v6’ so my database requires the migrations for v7 and v740.

What's strange is I installed from Marketplace so why was 6.0 installed and not 7.4.0? The MarketPlace offer specifically references 7.4.0.

Please advise how to apply the migrations to the production database?

Thanks very much for your attention on this – it is much appreciated.

Best regards, Adrian

Adrian Spear Technical Director

[cid:li-logo-codetwo-email-s_1e9e43da-653e-4bd9-b9f6-44776db87678.png]https://www.linkedin.com/in/adrianspear [cid:fs_logo_sml_r_10348fea_f68702a0-1fe9-4c67-b668-7f0690c7ccf8.png] https://fidrasoft.com/

@.**@.> | https://teamfolio.co.uk [cid:emailsignature_banner_s_eab6ce00-226a-4e79-808a-19abb64346e8.png] https://teamfolio.co.uk NOTE: This message may contain confidential and/or legally privileged information. If it has been sent to you in error, please reply immediately to advise the sender of the error and then destroy this message, any copies of this message and any printout of this message. If you are not the intended recipient of the message, any unauthorized dissemination, distribution or copying of the material in this message and any attachments to the message, is strictly forbidden. From: SANTHOSH B @.> Sent: Wednesday, November 29, 2023 12:06 AM To: Azure/Commercial-Marketplace-SaaS-Accelerator @.> Cc: Adrian Spear @.>; Mention @.> Subject: Re: [Azure/Commercial-Marketplace-SaaS-Accelerator] WebNotificationUrl Admin Setting Missing (Issue #610)

@adrian-spearhttps://github.com/adrian-spear which version are you using? latest 7.4.0? in that case then should see the setting coming in from the database,

— Reply to this email directly, view it on GitHubhttps://github.com/Azure/Commercial-Marketplace-SaaS-Accelerator/issues/610#issuecomment-1830974162, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATBYULLQSAN2ZVTPDNWYXKDYGZ35ZAVCNFSM6AAAAAA754IZ7OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZQHE3TIMJWGI. You are receiving this because you were mentioned.Message ID: @.**@.>>

santhoshb-msft commented 7 months ago

Hi @adrian-spear if you manually deployed then that means the db changes wont go in. I recommend following update instructions here. please be mindful any custom app changes will go away. if you just want to update db then use entity migrations commands here. Just a recommendation: Please always test in your test env before applying in active instance.

adrian-spear commented 7 months ago

Hi,

Thanks – no custom app changes and not planning on any at this time so I think we’re good.

I had found the powershell update info you recommend but wasn’t sure if this would apply the database changes – you indicate it will so I’ll go down that route.

We have not gone live with any marketplace offers yet so there’s nothing to lose if we have to tear down the accelerator and start again but rest assured a staging environment will be established before any offer go-lives take place 😊

All the best,

Adrian Spear Technical Director

[cid:li-logo-codetwo-email-s_1e9e43da-653e-4bd9-b9f6-44776db87678.png]https://www.linkedin.com/in/adrianspear [cid:fs_logo_sml_r_10348fea_f68702a0-1fe9-4c67-b668-7f0690c7ccf8.png] https://fidrasoft.com/

@.**@.> | https://teamfolio.co.uk [cid:emailsignature_banner_s_eab6ce00-226a-4e79-808a-19abb64346e8.png] https://teamfolio.co.uk NOTE: This message may contain confidential and/or legally privileged information. If it has been sent to you in error, please reply immediately to advise the sender of the error and then destroy this message, any copies of this message and any printout of this message. If you are not the intended recipient of the message, any unauthorized dissemination, distribution or copying of the material in this message and any attachments to the message, is strictly forbidden. From: SANTHOSH B @.> Sent: Tuesday, December 5, 2023 10:14 PM To: Azure/Commercial-Marketplace-SaaS-Accelerator @.> Cc: Adrian Spear @.>; Mention @.> Subject: Re: [Azure/Commercial-Marketplace-SaaS-Accelerator] WebNotificationUrl Admin Setting Missing (Issue #610)

Hi @adrian-spearhttps://github.com/adrian-spear if you manually deployed then that means the db changes wont go in. I recommend following update instructions herehttps://github.com/Azure/Commercial-Marketplace-SaaS-Accelerator/blob/main/docs/Installation-Instructions.md#update-to-a-newer-version-of-the-saas-accelerator. please be mindful any custom app changes will go away. if you just want to update db then use entity migrations commands herehttps://learn.microsoft.com/en-us/ef/core/managing-schemas/migrations/applying?tabs=dotnet-core-cli#idempotent-sql-scripts. Just a recommendation: Please always test in your test env before applying in active instance.

— Reply to this email directly, view it on GitHubhttps://github.com/Azure/Commercial-Marketplace-SaaS-Accelerator/issues/610#issuecomment-1841706410, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATBYULK2LGTFYXRIIMBBVK3YH6MCRAVCNFSM6AAAAAA754IZ7OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBRG4YDMNBRGA. You are receiving this because you were mentioned.Message ID: @.**@.>>

santhoshb-msft commented 7 months ago

Sounds good! Closing this for now. Please feel free to reopen or open a new issue as needed.