Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

Activenet registration #1297

Open
wants to merge 17 commits into
base: 8.x-1.x
Choose a base branch
from

Conversation

mattshoaf
Copy link
Contributor

@mattshoaf mattshoaf commented Sep 13, 2018

Make sure these boxes are checked before asking for review of your pull request - thank you!

If there is a new feature, please create PR against 8.x-1.xx-dev branch.

If this is a bug fix - use 8.x-1.x branch. We'll tag for release if the bug is critical asap or tag for release next bug fix release until critical issue arrived.

Steps for review

  • Enable the activenet and activenet_registration modules.
  • Enter your API information at /admin/config/services/activenet.
  • Enter the Base URL for the Active Communities domain at /admin/config/services/activenet/registration.
  • Add a ActiveNet Registration field to a node type.
  • Create a new page from that content type, and enter the desired query information to the FlexReg Registrations widget.
  • View the page and if you have matching registrations they will be displayed along with a link to the activenet site.

General checks

  • All coding styles are fulfilled and there are no any issues reported by CodeSniffer CI.
    CI code sniffer errors
  • All tests are running and there are no failed tests reported by CI.
    Behat test results
  • Documentation has been updated according to PR changes.
  • Steps for review have been provided according to PR changes.
    Steps for review
  • Make sure you've provided all necessary hook_update_N to support upgrade path.
  • Make sure your git email is associated with account on drupal.org, otherwise you won't get commits there.
    drupal.org email
  • If you would like to get credits on drupal.org, check documentation.

Issue created at drupal.org https://www.drupal.org/project/openy/issues/2999484

Thank you for your contribution!

@gundevel
Copy link
Collaborator

Can one of the admins verify this patch? Use "o+k to test" or ''t+est this please" for manual build execution.

@AlexNetman
Copy link
Contributor

retest this please

@podarok
Copy link
Contributor

podarok commented Oct 16, 2018

ok to test

@podarok podarok added this to the 1.14 milestone Oct 16, 2018
@gundevel
Copy link
Collaborator

Build comment file:

Environment Link
Fresh OpenY installation - Rose theme http://openy.cibox.tools/build1060
Fresh OpenY installation - Lily theme http://lily.openy.cibox.tools/build1060
Upgraded(upgrade path) installation http://upgrade.openy.cibox.tools/build1060
Installation process http://install.openy.cibox.tools/build1060/install.php

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/1060/

@ddrozdik ddrozdik added the PR: Needs Work Unfinished task. Issues still there label Jan 16, 2019
@gundevel
Copy link
Collaborator

Can one of the admins verify this patch? Use "o+k to test" or ''t+est this please" for manual build execution.

5 similar comments
@gundevel
Copy link
Collaborator

gundevel commented Jun 4, 2020

Can one of the admins verify this patch? Use "o+k to test" or ''t+est this please" for manual build execution.

@gundevel
Copy link
Collaborator

gundevel commented Jun 5, 2020

Can one of the admins verify this patch? Use "o+k to test" or ''t+est this please" for manual build execution.

@gundevel
Copy link
Collaborator

Can one of the admins verify this patch? Use "o+k to test" or ''t+est this please" for manual build execution.

@gundevel
Copy link
Collaborator

Can one of the admins verify this patch? Use "o+k to test" or ''t+est this please" for manual build execution.

@gundevel
Copy link
Collaborator

Can one of the admins verify this patch? Use "o+k to test" or ''t+est this please" for manual build execution.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR: Needs Work Unfinished task. Issues still there
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants