auvzhcet / auvzhcet_web

https://auvzhcet.netlify.com
http://mtsauvzhcet.com
0 stars 16 forks source link

Double Hull changed to SEA 5.0 #57

Closed hamid17amu closed 2 years ago

netlify[bot] commented 2 years ago

Deploy Preview for gallant-curie-a2595e ready!

Name Link
Latest commit 03019e93c17a6bf09f2102eb8d5df80d853a8ea7
Latest deploy log https://app.netlify.com/sites/gallant-curie-a2595e/deploys/62a875b4d3d50f000826f284
Deploy Preview https://deploy-preview-57--gallant-curie-a2595e.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

hamid17amu commented 2 years ago

I was told to do so...

On Tue, Jun 14, 2022 at 5:38 PM Prajjwal Nijhara @.***> wrote:

@.**** requested changes on this pull request.

Why did you resize the images?

— Reply to this email directly, view it on GitHub https://github.com/auvzhcet/auvzhcet_web/pull/57#pullrequestreview-1005740814, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJ3TGN7ZU5MXADMLPXNAQG3VPBY3BANCNFSM5YXNGBRA . You are receiving this because you authored the thread.Message ID: @.***>

pnijhara commented 2 years ago

@hamid17amu before pushing the changes did you run the website code in your system?

hamid17amu commented 2 years ago

yes, I did.

On Tue, Jun 14, 2022 at 5:43 PM Prajjwal Nijhara @.***> wrote:

@hamid17amu https://github.com/hamid17amu before pushing the changes did you run the website code in your system?

— Reply to this email directly, view it on GitHub https://github.com/auvzhcet/auvzhcet_web/pull/57#issuecomment-1155101286, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJ3TGN477SJIEKTUV2XC4HTVPBZMXANCNFSM5YXNGBRA . You are receiving this because you were mentioned.Message ID: @.***>

pnijhara commented 2 years ago

So it is a good practice to always attach the expected output screenshot. Are you getting the expected output based on your changes? Also, it is not a good practice to close the PR in the middle of a conversation.