search

integr8ly / tutorial-web-app

Solution Explorer provides the front door into the Integreatly initiative. It hosts the various Solution Patterns, as well as providing a dashboard of installed applications/products/services.
Apache License 2.0
36 stars 54 forks source link

fix: replace mdc with ups #525

Closed secondsun closed 4 years ago

secondsun commented 4 years ago

Motivation

This PR is part of adding UPS to the solution explorer for integr8ly and removing MDC.

What

This PR removes MDC and adds code for UPS. It replaces it in the mock data as well as adds hooks for the UPS service to be displayed.

Verification Steps

  1. Deploy integr8ly somewhere
  2. Modify the mobile-service-broker to use this image : quay.io/secondsun/managed-service-broker:master (a PR of msb is forthcoming)
  3. You can run this PR with the command line call OPTIONAL_PROVISION_SERVICES=unifiedpush OPENSHIFT_HOST=$YOUR_OPENSHIFT_HOST yarn start:dev (this will need to be added to the i8ly installer to start the solution explorer with this link)
  4. Navigate to localhost:3006 and you should see the push service link.

Checklist:

Progress

secondsun commented 4 years ago

Tell me the words and I'll make it happen :)

"mobile push messaging service" or "Provides messaging services for Android and iOS"?

Summers Pittman

Phone:404 941 4698 Java is my crack.

On Thu, Nov 21, 2019 at 9:44 AM Tiffany Nolan notifications@github.com wrote:

@tiffanynolan commented on this pull request.

In src/product-info.js https://github.com/integr8ly/tutorial-web-app/pull/525#discussion_r349123725 :

@@ -47,11 +47,11 @@ export default { primaryTask: 'Design APIs', description: 'The Apicurito RESTful API visual designer.' },

  • mdc: {
  • prettyName: 'Mobile Developer Console',
  • gaStatus: 'preview',
  • primaryTask: 'Create mobile apps',
  • description: 'Allows you register your mobile app and associate it with Mobile Developer Services.'
  • unifiedpush: {
  • prettyName: 'Push Notification Service',
  • gaStatus: 'GA',
  • primaryTask: 'Mobile Push Messaging',
  • description: 'Provides APNS and FCM messaging services for mobile applications.'

Can we make this description more user friendly? I'm not sure general users would know what APNS or FCM mean. Happy tow work with you on this.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/integr8ly/tutorial-web-app/pull/525?email_source=notifications&email_token=AASNJ6D4DBO4Q6G54E62TCLQU2NC7A5CNFSM4JPULJTKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCMQQ7SA#pullrequestreview-320933832, or unsubscribe https://github.com/notifications/unsubscribe-auth/AASNJ6CZ66SSQ6GKSO4HKSTQU2NC7ANCNFSM4JPULJTA .

secondsun commented 4 years ago

@tiffanynolan The RHPDS instance has been retired, but you seemed to be ok with the change minus some word smithing. If you can let me know what would be a good line for that I'll update the PR and merge so we can get on with starting a release to update the operator and install script.

tiffanynolan commented 4 years ago

@secondsun Thanks for the suggested description improvements. I like the second one better. It's simple and to the point. "Provides messaging services for Android and iOS"