Skip to content

napi-kupak.js.org #9887

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

napi-kupak.js.org #9887

wants to merge 1 commit into from

Conversation

mcnagynorbi
Copy link
Contributor

@mcnagynorbi mcnagynorbi commented Jun 21, 2025

The site content is a daily message page powered by JS

@MattIPv4 MattIPv4 changed the title napi-kupak napi-kupak.js.org Jun 21, 2025
Copy link
Member

@MattIPv4 MattIPv4 left a comment

Choose a reason for hiding this comment

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

👋 Please could you explain how this is related directly to the JavaScript ecosystem/community? Why is this something a JavaScript developer specifically would use more-so than anyone else?

@MattIPv4 MattIPv4 added the awaiting response This PR is waiting for a response from the requester label Jun 21, 2025
@mcnagynorbi
Copy link
Contributor Author

mcnagynorbi commented Jun 22, 2025

Since there are a couple of personal portfolio pages in the list I think a currently not too JS-y page is not that big of a problem.
My plan is to make this into an integrable JS widget.

@mcnagynorbi mcnagynorbi requested a review from MattIPv4 June 22, 2025 18:27
@MattIPv4
Copy link
Member

We no longer accept personal portfolios, they are grandfathered from before our new rules on content being directly related to the JS ecosystem.

@MattIPv4
Copy link
Member

I would recommend re-opening your request once you've got that JS widget built, so that it is demonstrably relevant to JS developers specifically.

@mcnagynorbi
Copy link
Contributor Author

I edited the repo to have a JS developer friendly demo and integrable script.
I'm gonna extend the list and work on the rest later since I have a big list from earlier but needs to be checked and reconsidered one by one.

@MattIPv4
Copy link
Member

The site content is a daily message page powered by JS

Live Online Visitor Count

You seem to have completely changed what your site is in Nagynorbii/napi-kupak@719ba1f... I am not convinced this is a legitimate request at all.

@indus recommend to close.

@mcnagynorbi
Copy link
Contributor Author

mcnagynorbi commented Jun 22, 2025

Yes I did. At first it only displayed the message, now it displays the code as well.
Also I copied the code template from my other project :)

@mcnagynorbi
Copy link
Contributor Author

mcnagynorbi commented Jun 22, 2025

I am fully aware of your rules, and I agree that it is important that the projects that are approved are really linked to the JS ecosystem. It is a rational and long term sustainable approach that I appreciate.

My current project is still in a very rudimentary stage - it's not a complete, finished project, but a foundation that I want to develop step by step. The current demo is just meant to show a minimal working version of the direction I want to go in: towards an embeddable JS widget.

js.org is very useful for developers like me. It makes a big contribution to making a project more visible and making a bigger impression - especially in the early stages - a hub where developers' projects come together (similar to npm).

The dataset that I want to integrate into this tool has been waiting for a better fate on my computer for years now. This demo (which I may have put together a bit hastily) is an important step for me to make it a full-fledged tool for JS developers in the future.

Of course, I completely understand if you don't want to merge the pull request for now and would rather wait until the project takes a more mature, clearly JS-centric form.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting response This PR is waiting for a response from the requester
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants