-
Notifications
You must be signed in to change notification settings - Fork 34
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
Customize tagged view in patterns website #113
Comments
Can I clarify what this is about?
|
(1) @osamajandali to investigate and see what can be done, but hasn't started yet. If the above is not possible will re-discuss |
There is a risk that this might not be done before COP, to be confirmed next week |
So the solution turns out to be very simple 😄 |
@osamajandali thanks that's good news I guess. can you do this before cop? cc'ing @zanete so she is aware |
@osamajandali can you clarify that this is something you will do or you provided an example for @holanita so that she (or someone else in the team) can do it? |
@holanita so I think the ask her is for Bill or someone from the patterns team to go through all the existing patters and add a short description at the top like @osamajandali showed how in here: https://github.com/Green-Software-Foundation/green-software-patterns/blob/main/docs/catalog/ai/serverless-model-development.md?plain=1#L6 @osamajandali can't do this, only the people who know the patterns can do this. |
@dubrie @greenhsu123 do you have capacity to add a short description to each pattern as described above? alternatively, if you could list the ones you can take care of, we could ask the team tomorrow to handle the rest. |
Sure, I can do the cloud ones. |
Just to clarify from my perspective, we have a description section in each of the patterns already. Is the ask to create a second, shorter, description that goes in the front matter section? Is it possible to use the description as it is today or do we want to re-write new ones for every pattern? |
@dubrie (and @osamajandali please correct me if I'm wrong but) I understand that it's difficult to use the existing description in the way is needed because it's part of the whole text and not placed as a tag above it. |
I had the same understanding @zanete. if this is just a copy-paste job, I can do this myself if no one else has time for this. |
This has been discussed in today's team meeting. We'll go for the copy-paste option. I will do this tomorrow. |
Okay so I've just submitted a PR for AI and WEB, but this seems a bit more tricky than expected.
|
@holanita ah... i am in Asia at the moment, it will have to wait till tomorrow am for me! could you make a start from the top? and i can catch up from the bottom when i wake up tomorrow :) |
No worries, I've just done Cloud too. @dubrie for Cloud I have used the original descriptions in most cases. Some have special characters like colons or urls, so I have removed some sentences from those descriptions. (I forgot that I should have committed to dev from the beginning so most of them are in a new branch sorry. I had 1 accidental commit in dev and couldn't find away to delete it. Can you do it form pls?) |
Thanks for reviewing the PR @dubrie. Just checked all the tags on the website everything seems fine. |
Customize the tagged page view:
The text was updated successfully, but these errors were encountered: