Provides a content pane to render a entityform by its bundle name

screenshot

git clone --branch 7.x-1.x http://git.drupal.org/sandbox/rodrigoaguilera/2621170.git entityform_pane

https://www.drupal.org/sandbox/rodrigoaguilera/2621170

Intended Drupal core version 7.x

CommentFileSizeAuthor
00000026.png27.52 KBrodrigoaguilera
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rodrigoaguilera created an issue. See original summary.

PA robot’s picture

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

rakesh.gectcr’s picture

Status: Needs review » Needs work

Please run review on http://pareview.sh/

Please fix the following errors

FILE: /var/www/drupal-7-pareview/pareview_temp/entityform_pane.info
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
1 | ERROR | "description" property is missing in the info file
----------------------------------------------------------------------

See more in following link
http://pareview.sh/pareview/httpgitdrupalorgsandboxrodrigoaguilera2621170git

rodrigoaguilera’s picture

Thank you!

issues addresed.

rodrigoaguilera’s picture

Status: Needs work » Needs review
gauravsood91’s picture

You should provide a Git repository link in the Project page.

gauravsood91’s picture

Status: Needs review » Needs work
rodrigoaguilera’s picture

Issue summary: View changes
Status: Needs work » Needs review

like this?
I think i don't quite understand what you are referring to

gauravsood91’s picture

Individual user account
Yes: Follows guidelines for individual user accounts.
No duplication
Could you elaborate why this should not fit under the Entity Form module?
Master Branch
Yes: Follows the guidelines for master branch.
Licensing
Yes: Follows the licensing requirements.
3rd party assets/code
Yes: Follows the guidelines for 3rd party assets/code.
README.txt/README.md
Yes: Follows the guidelines for in-project documentation and/or the README Template.
Code long/complex enough for review
Yes: Follows the guidelines for project length and complexity.
Secure code
Yes: Meets the security requirements
Coding style & Drupal API usage
Yes: It follows code and API guidelines.

adam_’s picture

Status: Needs review » Reviewed & tested by the community

Looks good!

Code quality looks good. Enough code to review and test. Styling/Spacing etc looks great. Follows licensing rules. README.txt is present. No major security holes or problems apparent.

Marking as RBTC.

DamienMcKenna’s picture

Status: Reviewed & tested by the community » Fixed

Thanks for your contribution, Rodrigo!

I updated your account so you can promote this to a full project and also create new projects as either a sandbox or a "full" project.

Here are some recommended readings to help with excellent maintainership:

You can find lots more contributors chatting on IRC in #drupal-contribute. So, come hang out and stay involved!

Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

Thanks to the dedicated reviewer(s) as well.

rodrigoaguilera’s picture

Thanks to everyone involved!

I'll update the code as soon as possible and release other projects with the same quality :)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.