-
Notifications
You must be signed in to change notification settings - Fork 13
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
Add Deployment badge to README #11
Comments
Can't do but hope I'll get to learn if someone does. |
I've done this a couple of times. I had to grant read access to the repositories first though as the image was being blocked. @Andreae4w I don't have organization-level access to change this. Are you able to? I've attached a screenshot of the setting we need to change. |
@gsambrotta @ConradParker @adityaraute I managed to update the settings from Azure. Thanks a lot for letting me know! |
Yeah, no. Sorry. I am still not sure if I know enough to solve this issue. Would request someone else to take it up. |
Hi @adityaraute if you need any help, we can guide you on working on this issue. |
Yes do not wish to work on it, I will observe from the sidelines. 😄 |
Isn't this as simple as adding the following line to the README.md file?
|
Is it? I tried, didn't work. |
Maybe I am misunderstanding the requirement here, but as @ConradParker pointed out the image is blocked and we need some access to see the actual status in the forked repository so that it would show the status. But after that access if granted isn't this the code change that this issue will need to resolve it? |
@sachinh19 you understood the requirement perfectly and yes, it should work adding just that line to the readme file. |
@gsambrotta Yes, I can confirm that. :) |
Now that the settings in azure are done, just adding that line to the README file should do the trick. I would suggest one of the owners of the repository to take this change up since there is a good chance that you folks will not see the error that we can see in @adityaraute's screenshot |
I can confirm the badge is still not showing. Sorry guys, we were hoping this was an easy one for starting out. I'll take a look at the permissions again. Leave it with me. |
@ConradParker @sachinh19 @adityaraute sorry guys, I have just noticed it did not take my change of settings on Azure. I have just corrected it and should be fine now. |
Thanks @Andreae4w , It's still not working for me so I'll keep investigating. |
thank you @ConradParker and everybody that helped! 🎉 |
Wait, it's done? I don't see a linked PR. |
Ah, right. here it is: #17 |
Add this cod to the README.md to show the status of the production deployment
Image URL
https://dev.azure.com/E4W/Elliotforwater/_apis/build/status/ElliotForWater.efw-webapp?branchName=main
The text was updated successfully, but these errors were encountered: