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

3 תרבות אירגונית #6

Merged
merged 17 commits into from
Aug 15, 2022
Merged

Conversation

Liadredhat
Copy link
Contributor

הפוסט הראשון שלי בגיט!
הכנסתי מידע שהצלחתי לאסוף. אשמח אם תעברו על זה.
תיקונים ושיפורים יתקבלו בברכה

@Liadredhat Liadredhat linked an issue Jul 28, 2022 that may be closed by this pull request
github-actions bot pushed a commit that referenced this pull request Jul 28, 2022
@bennypowers
Copy link
Member

bennypowers commented Jul 28, 2022

@TomerFi While 11ty did build the page, github pages cannot load it: https://redhat-israel.github.io/posts/%D7%AA%D7%9B%D7%99%D7%A8%D7%95%20%D7%90%D7%AA%20%D7%A8%D7%93%20%D7%94%D7%90%D7%98/ perhaps GH-pages struggles with unicode urls? If that's the case we can solve by renaming the post file with ascii chars. Please coordinate with @Liadredhat because she's still working on the content.

it builds and loads fine locally, so I suspect it is indeed a ghp thing
Screen Shot 2022-07-28 at 11 36 44

Copy link
Member

@ilan-pinto ilan-pinto left a comment

Choose a reason for hiding this comment

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

The article is great, but there are no images.
I suggest adding more images

@TomerFi
Copy link
Member

TomerFi commented Jul 28, 2022

@Liadredhat
אחלה פוסט - את יכולה לראות אותו במסגרת הפול ריקווסט בלינק הבא:
https://redhat-israel.github.io/red-hat-israel-site/pr-previews/pr-6/posts/%D7%AA%D7%9B%D7%99%D7%A8%D7%95%20%D7%90%D7%AA%20%D7%A8%D7%93%20%D7%94%D7%90%D7%98/

אחרי שימורגג', הלינק המפובלש אמור להיות:
https://redhat-israel.github.io/red-hat-israel-site/posts/%D7%AA%D7%9B%D7%99%D7%A8%D7%95%20%D7%90%D7%AA%20%D7%A8%D7%93%20%D7%94%D7%90%D7%98/

@bennypowers
זה לא רק הפוסט של ליעד שלא עובד, שום פוסט לא עובד.
נראה שהלינקים שנוצרים הם לא יחסיים.
כנס ללינק של תצוגת הדמה:
https://redhat-israel.github.io/red-hat-israel-site/pr-previews/pr-6/

כשמקליקים על הפוסט של ליעד, מתבצעת הפנייה ללינק:
https://redhat-israel.github.io/posts/%D7%AA%D7%9B%D7%99%D7%A8%D7%95%20%D7%90%D7%AA%20%D7%A8%D7%93%20%D7%94%D7%90%D7%98/

או כל פוסט אחר לצורך העניין:

https://redhat-israel.github.io/posts/fourthpost/

גם אם נתעלם מה-pr-previews/pr-6/ שהוא רלוונטי רק ל-preview, עדיין נראה שהלינקים לפוסטים הם לא יחסיים ולא כוללים את red-hat-israel-site/.

@bennypowers
Copy link
Member

@TomerFi

לא יחסיים

אכן נצטרך לחשוב על זה בaction של התצוגה מקדימה

  • או שנמיר את כל הלינקים היחסיים בci,
  • או שנשתמש בsubdomains (זה הפיתרון של netlify, אבל לא ברור לי אם מתאים לghp)

@TomerFi
Copy link
Member

TomerFi commented Jul 28, 2022

@bennypowers
אנחנו מתחננים custom cname?
אם כן - זה הופך להיות בעיה של ה-preview בלבד.
כי נכון לעכשיו - זה לא עובד לא ב-preview ולא ב-deployed.

ואם אנחנו לא מתכננים custom cname...
נכון לעכשיו אנחנו עובדים כ-project gh pages, מה שאומר שהלינק שלנו הוא:
https://redhat-israel.github.io/red-hat-israel-site
אולי נעבור את ה-repo הזה ולעבוד כ- user gh-pages ואז הלינק שלנו יהיה:
https://redhat-israel.github.io/
לדעתי זה יפתור לנו את הבעיה עבור ה-deployment, אבל לא עבור ה-preview.


EDIT

I think I got a workaround - will tag you in a new pr soon.

@TomerFi
Copy link
Member

TomerFi commented Jul 28, 2022

This needs rebasing.

@bennypowers bennypowers force-pushed the 3-תרבות-אירגונית branch from 66b54c7 to 6fc4ea8 Compare July 28, 2022 20:46
@bennypowers
Copy link
Member

rebased

github-actions bot pushed a commit that referenced this pull request Jul 28, 2022
@bennypowers
Copy link
Member

bennypowers commented Jul 28, 2022

כרגע התגים לא יכולים להכלליל unicode - English only, sorry

11ty/eleventy#1254 (comment)

bennypowers and others added 8 commits July 29, 2022 10:17
העתקתי תכנים חדשים על רד האט, היסטוריה ותרבות ארגונית.
 תיקנתי חלק של רד האט ישראל בהמשך להערה של יניב קאול
הוספתי תמונות כלליות וכאלה שקשורות לגיוס ואוסיף בהמשך עוד
הכנסתי תמונות בתיקיית img
ומנסה למשוך אותן לכאן
אישרתי שינויים במילה מתאפיינת עם שני יודים

Co-authored-by: Yaniv Kaul <ykaul@redhat.com>
הטמעתי את מרבית השינויים שהוצעו
Liadredhat and others added 3 commits July 29, 2022 10:17
tags must be a YAML array. tl;dr: separate each tag by a new line with two spaces and a hyphen
@bennypowers bennypowers force-pushed the 3-תרבות-אירגונית branch from c2d1d67 to 37d3d60 Compare July 29, 2022 07:17
github-actions bot pushed a commit that referenced this pull request Jul 29, 2022
Copy link
Member

@reihl reihl left a comment

Choose a reason for hiding this comment

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

אולי כדאי להשתמש בשמות אחרים לתמונות? WhatsApp_ נראה פחות מקצועי

github-actions bot pushed a commit that referenced this pull request Aug 1, 2022
@bennypowers
Copy link
Member

@reihl renamed the images

github-actions bot pushed a commit that referenced this pull request Aug 1, 2022
הוספתי תמונות אווירה
github-actions bot pushed a commit that referenced this pull request Aug 1, 2022
github-actions bot pushed a commit that referenced this pull request Aug 2, 2022
github-actions bot pushed a commit that referenced this pull request Aug 2, 2022
github-actions bot pushed a commit that referenced this pull request Aug 2, 2022
@bennypowers
Copy link
Member

אני רוצה למרגג את זה ביום ראשון ושנמשיך לעבוד עליו בPRים קטנים. הגיוני?

@bennypowers bennypowers enabled auto-merge (squash) August 12, 2022 11:40
@reihl
Copy link
Member

reihl commented Aug 12, 2022

/LGTM נראה לי בסדר לעשות Merge.
לגבי הנקודה השניה, אני חסיד גדול של Atomic commits

@bennypowers
Copy link
Member

נלמט (נראה לי ממש טוב) 😉

@bennypowers bennypowers merged commit 8365a6d into main Aug 15, 2022
@bennypowers bennypowers deleted the 3-תרבות-אירגונית branch August 15, 2022 10:48
github-actions bot pushed a commit that referenced this pull request Aug 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

תרבות אירגונית
7 participants