d3.js, currently for Drupal 7.x, changes the idea of a visualization from just a "chart", to an interactive series of charts that can be easily swapped out with use of the libraries module. While there are customization options, and views integration, the developer has the ability to create any new type of visualization with Drupal just by adding the library.

The module simply sends the desired php data straight to javascript. All the flexibility can lie with the javascript itself, giving the developer the ability to create any variation of visualizations.

Project page:
https://drupal.org/project/d3

Git repository:
git.drupal.org:project/d3.git

d3.js was promoted to a full project by another user, I would like to go through the approval process so that I have 'promote project' access on drupal as well.

Reviews of other projects:
http://drupal.org/node/2055867#comment-7800287
http://drupal.org/node/2053373#comment-7800225
http://drupal.org/node/2058943#comment-7800251
http://drupal.org/node/2059041#comment-7800325

More reviews of other projects:
http://drupal.org/node/2080721#comment-7824185
http://drupal.org/node/2077293#comment-7824813
http://drupal.org/node/2054613#comment-7824959

Comments

klausi’s picture

Assigned: Unassigned » klausi

I'll look at this now in the Project applications sprint

klausi’s picture

Assigned: klausi » Unassigned
Status: Active » Postponed (maintainer needs more info)

That situation is a bit unfortunate. I would suggest that you get in contact with that maintainer to talk about collaboration and making you a co-maintainer. If you cannot reach the maintainer(s) please follow the abandoned project process.

If that fails for whatever reason please get back to us and set this back to "needs review".

asherry’s picture

Status: Postponed (maintainer needs more info) » Needs review

I have heard back from the maintainer, and they are on board with the switch back.

My question for the project application team is can we do what batje is suggesting in the thread below? I would like to have that module promoted and the ownership switched to me, but also very important to me is that I get a legitimate project approved and gain access to promote projects on d.o.

https://drupal.org/node/2067393#comment-7774111

PA robot’s picture

Status: Needs review » Needs work

There are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpgitdrupalorgsandboxasherry1477334git

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.

asherry’s picture

Status: Needs work » Needs review

I have finished the review by pareview.sh. The only errors I cannot fix are the libraries .info files that it seems to be evaluating like module.info files. Libraries info files do not have a 'core' attribute, and the files array needs a [js] and [css] key.

I also had an issue with a long string that gets output to svg.
libraries/d3.tooltip/tooltip.js - line 55

This is much easier to have as a concatenated string, I'm wondering if I really have to condense it.

Thank you very much,
~Alan~

asherry’s picture

I have finished the review by pareview.sh. The only errors I cannot fix are the libraries .info files that it seems to be evaluating like module.info files. Libraries info files do not have a 'core' attribute, and the files array needs a [js] and [css] key.

I also had an issue with a long string that gets output to svg.
libraries/d3.tooltip/tooltip.js - line 55

This is much easier to have as a concatenated string, I'm wondering if I really have to condense it.

Thank you very much,
~Alan~

asherry’s picture

Issue tags: +PAreview: review bonus

Added review bonus tag.

asherry’s picture

Issue summary: View changes

Updated issue summary.

bappa.sarkar’s picture

bappa.sarkar’s picture

Issue summary: View changes

Updated issue summary.

asherry’s picture

@bappa.sarker - I'm not sure what you mean by 'add title [D7]'

I added the git repo in the issue description, and took out the .gitignore - I can see why that's not really all that useful for a normal drupal user who's downloading the module.

The pareview.sh script is commenting on the method names in one of the classes, but I'm overridding core views module methods. I guess I could rename the custom methods I have to camel case but I think it'd be better to remain consistent and not have two different naming conventions in the same class.

asherry’s picture

Title: d3.js » [D7] d3.js

@bappa.sarkar - nevermind on my question. Title is fixed.

klausi’s picture

Status: Needs review » Needs work

You already have commit access to the existing d3 project, sou you should push all your commmits to that project.

Then please delete your sandbox, so that there is no confusion anymore which project to use. Then change the git repository and link to project page here in the issue summary, so that we review the correct stuff. Please also correct the issue summary here, you are a co-maintainer now and have access.

clemens.tolboom’s picture

@klausi:

We must give @asherry maintainer and node ownership of http://drupal.org/project/d3 as it's very weird @batje was able to create the project. I tried it too and got a validation error.

Even the git log / attribution in @asherry repo should be incorporated. I guess @asherry could use some help with that part.

I contributed to @asherry project so am feeling said about all this :-/

clemens.tolboom’s picture

klausi’s picture

asherry already has maintainer access on the project, he is just not the project node owner, which is a minor detail and can be fixed with a webmaster issue if everybody agrees.

The git history can be merged, this should do it:

git clone YOURUSER@git.drupal.org:sandbox/asherry/1477334.git d3
cd d3
git remote add fullproject YOURUSER@git.drupal.org:project/d3.git
git fetch fullproject
git merge -s ours fullproject/7.x-1.x
git push fullproject 7.x-1.x

Note that this favors the changes in your sandbox, so in case of conflicts the version from the sandbox is used.

clemens.tolboom’s picture

@klausi thanks for the instruction @asherry can use :)

asherry’s picture

@klausi, the main issue is I was never granted access to promote projects to full projects. I wanted this module to go through the process so that I would gain access.

Also does this mean I lose the issue queue?

asherry’s picture

Issue summary: View changes

Updated issue summary.

asherry’s picture

Status: Needs work » Needs review

Sorry @klausi, I misunderstood. I made the changes you specified, those worked great. It should be now ready to start the process. I already have the ok by @batje and @greggles to change the ownership of the project - but I'm assuming that's done by a different d.o. admin. I'll take that issue up with them if need be.
https://drupal.org/node/2067393#comment-7774133

Thank you.

PA robot’s picture

Status: Needs review » Needs work

Link to the project page and git clone command are missing in the issue summary, please add them.

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

klausi’s picture

Status: Needs work » Needs review

Poor robot, she is trained to detect sandboxes only ;-)

asherry’s picture

I suppose that makes sense, probably not all that often that people apply to have an already promoted project... promoted. :-)

klausi’s picture

Assigned: Unassigned » kscheirer
Status: Needs review » Reviewed & tested by the community
Issue tags: -PAreview: review bonus

manual review:

  1. d3_libraries_info_alter(): why is that needed? Please add a comment.
  2. "variable_get('d3_graphapi_default_library', 'forcedirected')": all variables defined by your module need to be removed in hook_uninstall().

But otherwise looks RTBC to me. Removing review bonus tag, you can add it again if you have done another 3 reviews of other projects.

Assigning to kscheirer as he might have time to take a final look at this.

asherry’s picture

Thanks - I added the comment for the info_alter, and the uninstall hook.

asherry’s picture

Issue summary: View changes

Changed the links for the project, we are now referencing the full project.

asherry’s picture

Issue tags: +PAreview: review bonus

I did more reviews of other projects, glad to help out with others going through the same process!

kscheirer’s picture

Assigned: kscheirer » Unassigned
Status: Reviewed & tested by the community » Fixed

I think the module looks great, no major issues found.

Thanks for your contribution, asherry!

I updated your account to let you 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 get 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.

----
Top Shelf Modules - Crafted, Curated, Contributed.

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

Anonymous’s picture

Issue summary: View changes

Added more reviews of other projects