emberjs / website

Source for emberjs.com
https://emberjs.com
Other
643 stars 1.33k forks source link

Simplify adding new companies to the website #3215

Open sivakumar-kailasam opened 6 years ago

sivakumar-kailasam commented 6 years ago

For a new company to be added to https://emberjs.com/ember-users/ we require a PR. It might be easier if we could create an airtable form for these submissions & have an integration that'd automatically raise a PR to this repo.

kennethlarsen commented 6 years ago

There are possible solutions which need to be discussed:

1) Create an airtable form which triggers a bot to make a PR. This can definitely be done (however, a bit tricky since airtable still doesn't support notify new submissions via the API). But, we might end up with duplicated info that might desync.

2) Create an airtable form and use the airtable API directly in the ember app. There are a few issues that need to be resolved here. First is validation on submitted images which is not possible on airtable (size and format). This would require some manual work and we would need to figure out how to tell the submitter to re-submit with a proper image.

3) Create an airtable sheet and keep validation and form client side in the ember app. This way we use airtable as an API to submit and retrieve data but keep the form and validation in the app itself.

sivakumar-kailasam commented 6 years ago

Another way to look at approach 1 is for us to run an app that can be run periodically that could look up the company's name from airtable on the yml file & then create the PR based on it.

locks commented 5 years ago

@sivakumar-kailasam we have done work in this area, what's the current status?

sivakumar-kailasam commented 5 years ago

I haven’t tried anything but I think @kennethlarsen did

mansona commented 5 years ago

There is a new system on the https://github.com/ember-learn/ember-website site that should significantly improve this situation. Each company has its own MD file (like most of our other data) and it currently works well with a generator approach

@kennethlarsen we should chat about this if you're still planning to look into it πŸ‘

kennethlarsen commented 5 years ago

@mansona I didn't get far with this because there didn't seem to be an agreement on the approach. The only thing there seemed to be a consensus about is that it would be neat with a form directly on the website to make it easier for everyone to add their company.

mansona commented 5 years ago

@kennethlarsen how about a form that created a PR? That seems like something within the realms of possibility with the new system πŸ˜‚

kennethlarsen commented 5 years ago

@mansona That sounds just like the kind of wizardry we need! πŸ§™

mansona commented 5 years ago

πŸŽ‰ I'll add it to the list so πŸ˜‚ Let me know if you would like to pair on a solution to this.

Also if you (or anyone) could give me an indication of the priority of this that would be great. Do we have a feel for how much of a problem it is for people? We seem to get a steady stream as is πŸ€”