Introduction

Bynder Orbit integration module, providing seamless access to Orbit assets in your Drupal website. The Drupal integration will allow the authorised users to:

  • Import Assets from Orbit
  • Display image styles imported from Orbit

Project page

https://www.drupal.org/project/bynder_orbit

Dependencies

Module dependencies:

  • Media - The Media Browser is an essential part of the module.
  • jQuery Update - This module requires a recent version (1.7+) of jQuery in order to work.
  • Composer Manager - We use composer to manage library dependencies.

Requirements

In order to start using this module you'll first need a Bynder Orbit account and to generate your Oauth2 credentials.Have a look in the Before you get started section of the documentation for more details on how to get started.

Git command

git clone --branch 7.x-1.x https://git.drupal.org/project/bynder_orbit.git

Manual reviews of other projects

https://www.drupal.org/project/projectapplications/issues/2938477#comment-12459331
https://www.drupal.org/project/projectapplications/issues/2938337#comment-12459458
https://www.drupal.org/project/projectapplications/issues/2920677#comment-12459491

CommentFileSizeAuthor
Screen Shot 2018-01-05 at 13.40.46.png908.61 KBjfs.csantos
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jfs.csantos created an issue. See original summary.

jfs.csantos’s picture

Issue summary: View changes
jfs.csantos’s picture

Issue summary: View changes
jfs.csantos’s picture

Issue tags: +PAreview: review bonus
jfs.csantos’s picture

Issue summary: View changes
jfs.csantos’s picture

Issue summary: View changes
jfs.csantos’s picture

Reviewing coding standards and indenting at the moment.

Mario Steinitz’s picture

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

Please fix these coding standard issues first before you set this issue to "Needs review". Especially indentations within your files. Currently they mess up any automated tests, so that best practice checks are too hard to isolate.

Some feedback anyway:

Review of the 7.x-1.x branch (commit 7d4ca8e):

I also altered your git checkout command.

jfs.csantos’s picture

Status: Needs work » Needs review
jfs.csantos’s picture

@Mario, thanks for the initial review, I've done (I think) all the changes you mentioned together with other formatting and Drupal standards changes. Let me know if you can have a look

sleitner’s picture

Status: Needs review » Closed (outdated)

@jfs.csantos the project is marked as obsolete and there is a similar project you are working on https://www.drupal.org/project/bynder