geeklearningio / gl-vsts-tasks-ghost

Bring some devops process to Ghost Pro with VSTS
MIT License
3 stars 0 forks source link

Flow errors and diagnostic information to the logs #3

Open sandorfr opened 6 years ago

sandorfr commented 6 years ago

Are you reporting a bug or a feature request ?

Description

We must flow details to the logs:

jessehouwing commented 6 years ago

I understand this would be easier using an API, but the current error messages returned by ghost are relatively easy to capture and flow back. It shouldn't require screenshots.

Errors

Warnings

On Wed, 4 Jul 2018, 02:26 Sandor, notifications@github.com wrote:

Are you reporting a bug or a feature request ?

  • Bug
  • Feature request

Description

We must flow details to the logs:

  • upload failed
  • login failed
  • invalid file

Solution

Ghost doesn't expose a public API to upload themes. An solution to move this forward would be to make a PR to the official repo to add required APIs.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/geeklearningio/gl-vsts-tasks-ghost/issues/3, or mute the thread https://github.com/notifications/unsubscribe-auth/AD-uS2etQMjycGb0GQ718Xsld8I259JKks5uDAutgaJpZM4VByy7 .

geeklearningbot commented 6 years ago

Yes I’m on it :) thanks for being the first alpha tester.

jessehouwing commented 6 years ago

Thanks for building this. The Ui automation approach is pretty interesting.

On Wed, 4 Jul 2018, 13:28 geeklearningbot, notifications@github.com wrote:

Yes I’m on it :) thanks for being the first alpha tester.

Get Outlook for iOShttps://aka.ms/o0ukef


From: Jesse Houwing notifications@github.com Sent: Wednesday, July 4, 2018 6:41:30 PM To: geeklearningio/gl-vsts-tasks-ghost Cc: Subscribed Subject: Re: [geeklearningio/gl-vsts-tasks-ghost] Flow errors and diagnostic information to the logs (#3)

I understand this would be easier using an API, but the current error messages returned by ghost are relatively easy to capture and flow back. It shouldn't require screenshots.

Errors

  • username/password incorrect
  • invalid theme file

Warnings

  • theme was updated/uploaded successfully, but wasn't activated.

On Wed, 4 Jul 2018, 02:26 Sandor, notifications@github.com wrote:

Are you reporting a bug or a feature request ?

  • Bug
  • Feature request

Description

We must flow details to the logs:

  • upload failed
  • login failed
  • invalid file

Solution

Ghost doesn't expose a public API to upload themes. An solution to move this forward would be to make a PR to the official repo to add required APIs.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/geeklearningio/gl-vsts-tasks-ghost/issues/3, or mute the thread < https://github.com/notifications/unsubscribe-auth/AD-uS2etQMjycGb0GQ718Xsld8I259JKks5uDAutgaJpZM4VByy7

.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub< https://github.com/geeklearningio/gl-vsts-tasks-ghost/issues/3#issuecomment-402408177>, or mute the thread< https://github.com/notifications/unsubscribe-auth/AR20c9kDfy6VXukAbS2qfKbFaStmagXJks5uDH-6gaJpZM4VByy7

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/geeklearningio/gl-vsts-tasks-ghost/issues/3#issuecomment-402450979, or mute the thread https://github.com/notifications/unsubscribe-auth/AD-uS8H7p8L5EYaYUEaLVodQ4hJBV9dSks5uDKbEgaJpZM4VByy7 .

sandorfr commented 6 years ago

Login errors are now flowed back to the console.