Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UX shows "not yet deployed" for a new local deployment which has been associated with a deployment on connect #2349

Open
dotNomad opened this issue Oct 3, 2024 · 7 comments

Comments

@dotNomad
Copy link
Collaborator

dotNomad commented Oct 3, 2024

Reported as "weird" from Neil, during demo.

Can we adjust the wording to indicate that an update has not been deployed from here but that this project has been deployed?

@dotNomad dotNomad added the design label Oct 3, 2024
@dotNomad
Copy link
Collaborator Author

dotNomad commented Oct 3, 2024

We do update the wording when associating a Deployment:

Image

We may need clarification on if the "Not Yet Deployed" wording is weird prior to the association or if the "Not Yet Updated" wording is weird.

@dotNomad
Copy link
Collaborator Author

dotNomad commented Oct 3, 2024

@nealrichardson would love your thoughts on this one 😄

@nealrichardson
Copy link

I don't think "Not Yet Updated" is useful information in this context. It makes me think something is wrong.

If you're saying "you or someone else has deployed this before, just not from this IDE on this machine", I'm not sure how much difference that makes to me, no matter how technically true that is.

If you're saying "you have unpublished changes", that would be true regardless of where I've published from before.

Why shouldn't this basically look the same as it does when I've previously published on this computer?

@dotNomad
Copy link
Collaborator Author

dotNomad commented Oct 4, 2024

All fantastic points.

Why shouldn't this basically look the same as it does when I've previously published on this computer?

Great question. Here is what is shown when a Deployment has been published successfully with Publisher:

Image

We get the timestamp from our Deployment record. We could use the same messaging and just not include the timestamp since it is not present in the Deployment record.

The "View Content" button is still around to have that "double check I pointed this to the right place" flow for those that want that, and the messaging becomes consistent to avoid the signaling of something different and possibly wrong.

@nealrichardson
Copy link

Could you get the "last deployment" timestamp from the Connect API? (Aside, and feel free to file this as YAGNI for now if you want: it's probably useful information to know whether the latest deployment on Connect matches the local deployment record, for other cases where someone else did the most recent deployment.)

Other aside: "Last Deployment Successful" feels redundant. I know you're distinguishing the case where the last (attempted) deployment failed, but all I want (as a user) from this space is to see the last time the content was updated on Connect.

@dotNomad
Copy link
Collaborator Author

dotNomad commented Oct 7, 2024

it's probably useful information to know whether the latest deployment on Connect matches the local deployment record, for other cases where someone else did the most recent deployment

Absolutely. We have set aside all potential work of syncing with the Connect server to keep the first release simple, but there are quite a few cases where we want to implement features exactly like this.

Thanks for the suggestion 😄

@dotNomad
Copy link
Collaborator Author

dotNomad commented Oct 7, 2024

From @sagerb we do make Connect API calls to make the association so we could write in the timestamp to the content record without building any syncing functionality.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants