Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge #98, #100, #101, #102, #103, #104, #107, #108, #113, #114, #115, #118, #119, #121, #122, #126 into 7.0 #127

Merged
merged 31 commits into from
May 5, 2017

Conversation

tyearke
Copy link
Contributor

@tyearke tyearke commented May 4, 2017

Description

This pull request merges #98, #100, #101, #102, #103, #104, #107, #108, #113, #114, #115, #118, #119, #121, #122, #126 from Open XDMoD 6.7 to 7.0.

This merge requires conflict resolution, which was done in merge commit 6c5bdeb.

File conflicts:

  • bin/xdmod-upgrade
  • configuration/portal_settings.ini
  • open_xdmod/modules/xdmod/build.json
  • package.json

jtpalmer and others added 30 commits April 17, 2017 10:03
…ngleton

Replace singleton composite logger
Remove script used to generate API docs
Modify xdmod source for php7 compatibility. Schema version changes requested by @jtpalmer are included.
Add xdmod-test-artifacts as a compuser dev dependency.
Fix the forgot password form so that it looks correct
The file included by the removed script tag is part of the App Kernels
module.
* Initial version of support for RFC-6901 JSON References

* Remove warning for duplicate data endpoint name

* Move local config dir handling into Configuration object

* Add documentation for JSON references and verification script

* Add component tests for Configuration, EtlConfiguration, RFC-6901 parsing

* PHPCS and extra whitespace cleanup
- Typing a '%' character in a form field contained within the
  MetricExplorer chart options menu would not have the desired effect. Instead,
  the whole menu would close and the '%' character would not be recorded in the
  field. The reason for this is because the doRelay handler for the
  chartOptionsMenu.keyNav currently always relays events regardless of whether
  or not the event is an actual keyNav event or not. This updates that handler
  to only call the handler in the case the event is for a keyNav event, thus '%'
  is no longer interpreted as 'left'. This version of the fix was pointed out by
  a code review comment by @jpwhite4.
* Bumped build_packager memory limit

- When executing a local deploy ( as opposed to pulling down the source from a
  git repo ) this script consistently runs out of memory. Bumping the limit to
  256M alleviates this issue.

* Removed Memory Limit for build_package.php script

- Updated per code review comment by @jtpalmer
- Fix for NavKey handlers being executed for non-NavKey key presses.
Add 'character set' specification for XRAS ingestors to ETLv1
…irst-update

Metric Explorer: Fixed errors not displaying if help graphic active
@tyearke tyearke added this to the v7.0.0 milestone May 4, 2017
@tyearke tyearke changed the base branch from xdmod6.6 to xdmod7.0 May 4, 2017 17:19
@tyearke tyearke changed the title Merge #100, #101, #104, #105, #106, #107, #109, #110, #112, #114, #116, #117, #118, #122, #126 into 7.0 Merge #98, #100, #101, #102, #103, #104, #107, #108, #113, #114, #115, #118, #119, #121, #122, #126 into 7.0 May 4, 2017
Conflicts:
	bin/xdmod-upgrade
	configuration/portal_settings.ini
	open_xdmod/modules/xdmod/build.json
	package.json
@tyearke tyearke merged commit e214bb0 into ubccr:xdmod7.0 May 5, 2017
@tyearke tyearke deleted the merges/6.7-2017-05-04 branch October 5, 2017 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants