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.

Sign up for GitHub

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

Jump to bottom

Add link to deployment details in GHA comment #144

Closed
zipofar opened this issue Aug 11, 2022 · 1 comment · Fixed by #146
Closed

Add link to deployment details in GHA comment #144

zipofar opened this issue Aug 11, 2022 · 1 comment · Fixed by #146
Assignees
@zipofar @antlu
Labels
community-new dev test Low risk issue that should be tested by developer only

Comments

@zipofar
Copy link
Contributor

zipofar commented Aug 11, 2022

Related issue UffizziCloud/preview-action#7
@zipofar zipofar mentioned this issue Aug 11, 2022
Closed
zipofar pushed a commit that referenced this issue Aug 11, 2022
@the38th
[#144] Add link to deployment details in GHA comment
285d17d
@zipofar zipofar linked a pull request Aug 11, 2022 that will close this issue
[#144] Add link to deployment details in GHA comment #146
Merged
antlu added a commit that referenced this issue Aug 19, 2022
@antlu
[#144] Delete excessive normalization
363abee
antlu added a commit that referenced this issue Aug 19, 2022
@antlu
[#144] Delete excessive normalization
4867035
antlu added a commit that referenced this issue Aug 19, 2022
@antlu
[#144] Delete excessive normalization
ff6d96e
antlu added a commit that referenced this issue Aug 19, 2022
@antlu
[#144] Delete excessive normalization
3bc31b3
antlu added a commit that referenced this issue Aug 19, 2022
@antlu
[#144] Delete excessive normalization
455a32f
@zipofar
Copy link
Contributor Author

zipofar commented Aug 19, 2022

HOW TO TEST:

Create preview from CLI with output = json uffizzi preview create dc.uffizzi.yaml --output json

Expected result (containers_uri field must be present):

{"id":"deployment-5152","url":"https://deployment-5152-p1-g9gu.app.uffizzi.com","containers_uri":"https://app.uffizzi.com//projects/3024/deployments/5152/containers"}

@moklidia moklidia added the dev test Low risk issue that should be tested by developer only label Aug 22, 2022
zipofar pushed a commit that referenced this issue Aug 22, 2022
@the38th @zipofar
[#144] Add link to deployment details in GHA comment
8c9a3b8
zipofar pushed a commit that referenced this issue Aug 22, 2022
@antlu @zipofar
[#144] Delete excessive normalization
fb5b1ed
@moklidia moklidia closed this as completed Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zipofar zipofar

@antlu antlu

Labels
community-new dev test Low risk issue that should be tested by developer only
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#144] Add link to deployment details in GHA comment UffizziCloud/uffizzi_cli
3 participants
@zipofar @antlu @moklidia