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

adding-updated design-to-how-it-works-section-akri #4928

Merged
merged 4 commits into from
Nov 29, 2023

Conversation

PRANJALRANA11
Copy link
Contributor

@PRANJALRANA11 PRANJALRANA11 commented Sep 17, 2023

Description

This PR fixes

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Signed-off-by: PRANJALRANA11 <[email protected]>
Signed-off-by: PRANJALRANA11 <[email protected]>
@PRANJALRANA11
Copy link
Contributor Author

@abhijeetgauravm thanks for helping

@l5io
Copy link
Contributor

l5io commented Sep 17, 2023

🚀 Preview for commit a709b32 at: https://6507157e851b2749b421947f--layer5.netlify.app

Copy link
Contributor

@abhijeetgauravm abhijeetgauravm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@Ghat0tkach
Copy link
Member

hey @PRANJALRANA11
Let's discuss it on the websites call.
I have already added your agenda as part of today's meeting minutes

https://docs.google.com/document/d/1XczAHXVe2FIWPqiF57ospJ43zw5cZQ7ui8mn39v5EvA/edit#heading=h.lohhtewfwima

Copy link
Member

@Mohith234 Mohith234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • These are not the actual screenshots that are required @PRANJALRANA11
  • Do check out the Akri integration and replace it with the existing images.

@PRANJALRANA11
Copy link
Contributor Author

But I think in website call meeting lee had said that these are good as there is no screenshot reference provided in spreadsheets

@PRANJALRANA11
Copy link
Contributor Author

and I think every integration have same screenshot

@leecalcote
Copy link
Member

Both of you fine gentlemen are correct, @Mohith234 and @PRANJALRANA11.

Having these screenshots, while not of Akri (which is the ideal - what the issue calls for) puts us in a better position that reusing the same screenshots that most of the rest of the pages have. We could merge this PR as a short-term stop gap until Akri-specific screenshots are taken.

@PRANJALRANA11
Copy link
Contributor Author

@leecalcote please merge this pr

@leecalcote
Copy link
Member

@PRANJALRANA11 when will you have appropriate screenshots?

@leecalcote
Copy link
Member

Instead of pressuring to merge the PR, how about you provide a genuine fix? Otherwise, remove the "fixes ####" in the PR description so that the issue doesn't close upon merge and unassign yourself from the issue, so that someone else has a chance to sign up to provide an accurate fix.

@PRANJALRANA11
Copy link
Contributor Author

@leecalcote but in above comment you have said that We could merge this PR as a short-term stop gap until Akri-specific screenshots are taken so that's why I was asking is there is a mistake in my understanding please tell

@leecalcote
Copy link
Member

Yes, you’re right. I’m also recognizing that when this PR is merged, so too will be the issue tracking the continued need for screenshots of the actual technology be closed.

Again, you’re asking for the PR to be merged, and still not answering my question, which is precisely the thing that gives me pause here. Your concern is with getting a PR merged and not with resolving the actual issue. This reinforces in my mind that you will not come back to fix the actual issue. So, I will ask again the same question that I just asked…

@PRANJALRANA11
Copy link
Contributor Author

@leecalcote I think I took that all wrong sorry for that but I was unable to find the Akri integration in the components of playground could you tell me where it is once I find it I just have to make a design with I am right

@l5io
Copy link
Contributor

l5io commented Nov 6, 2023

🚀 Preview for commit ec8e52c at: https://6548f7d39c2cf20f1d129d55--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Nov 12, 2023

🚀 Preview for commit 8b51026 at: https://65504b8d51f89631098cf905--layer5.netlify.app

@leecalcote leecalcote merged commit 824bf83 into layer5io:master Nov 29, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

7 participants