Skip to content

Add nifty story - #683

Merged
yuhattor merged 1 commit into
masterfrom
adding-nifty-logo
Dec 13, 2023
Merged

yuhattor merged 1 commit into
masterfrom
adding-nifty-logo

Conversation

@yuhattor

Copy link
Copy Markdown
Member

This pull request includes changes to the content/en/stories/_index.md, content/fr/stories/_index.md, and content/ja/stories/_index.md files. The changes add a new company, "nifty", to the list of companies in each of these files.

Addition of new company:

  • content/en/stories/_index.md: Added a new company "nifty" with its logo, video link, article link, and author names.
  • content/fr/stories/_index.md: Added the same company "nifty" with its corresponding details in the French version of the file.
  • content/ja/stories/_index.md: Included the "nifty" company details in the Japanese version of the file as well.

@yuhattor
yuhattor requested a review from a team as a code owner December 13, 2023 05:43
@yuhattor

Copy link
Copy Markdown
Member Author

@rashikawat If it's okay, please confirm and approve it (For approval, please give me a 👍 emoji)

@spier spier left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@yuhattor I think if you have evidence of the use of InnerSource practices, as well as confirmation from the org that they are happy to be listed, then that is all you need to publish a story.

Thank you for adding these stories from Japan!

@yuhattor

yuhattor commented Dec 13, 2023 •

Copy link
Copy Markdown
Member Author

@spier Thank you!
Then Nifty would have already fulfilled it. It has been approved by @rashikawat, Nifty's person in charge, and the article is full of InnerSource evidence :)

I will merge it

@yuhattor
yuhattor merged commit 70ea92c into master Dec 13, 2023
@yuhattor
yuhattor deleted the adding-nifty-logo branch December 13, 2023 13:39
@spier

spier commented Dec 13, 2023

Copy link
Copy Markdown
Member

And it is live 🥳

Screenshot 2023-12-13 at 14 44 58

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.

2 participants