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

[OYPD-519] Add alt text to the open category buttons on the sub category page. #620

Merged
merged 1 commit into from Jun 27, 2017
Merged

[OYPD-519] Add alt text to the open category buttons on the sub category page. #620

merged 1 commit into from Jun 27, 2017

Conversation

ivan-berezhnov
Copy link
Contributor

@ivan-berezhnov ivan-berezhnov commented Jun 26, 2017

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

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.

Thank you for your contribution!

Jira issue:

https://propeople-us.atlassian.net/browse/OYPD-519
Drupal.org issue: https://www.drupal.org/node/2889604

Steps to review:

  • go to frontpage
  • in the main menu click on "Program" and click on any category
  • in the list of subcategory find button with text "Open category"
  • check link have title attribute with node name as text in H2

@ivan-berezhnov ivan-berezhnov self-assigned this Jun 26, 2017
@ivan-berezhnov ivan-berezhnov added PR: Needs Review Needs someone review ( code ) PR: Needs Testing Manual testing is needed labels Jun 26, 2017
@ivan-berezhnov ivan-berezhnov added this to the Version 1.3 milestone Jun 26, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 27, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 27, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 27, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 27, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 27, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 27, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 27, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 27, 2017
@ymcatwincities ymcatwincities deleted a comment from AlexNetman Jun 27, 2017
@ymcatwincities ymcatwincities deleted a comment from ivan-berezhnov Jun 27, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 27, 2017
@Sanchiz Sanchiz added PR: Code Reviewed Code reviewed. Ready for QA and removed PR: Needs Review Needs someone review ( code ) labels Jun 27, 2017
@gundevel
Copy link
Collaborator

Build comment file:

===============

Environment Link
Fresh OpenY installation http://ci.openymca.org/build1744
Upgraded(upgrade path) installation http://upgrade.openy-dev.ffwua.com/build1744
Installation process http://install.openy-dev.ffwua.com/build1744/install.php

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://ci.openymca.org:8080/job/PR_BUILDER/1744/

@gundevel
Copy link
Collaborator

Decoupling report http://ci.openymca.org/build1744

@gundevel
Copy link
Collaborator

Accessibility report http://ci.openymca.org/build1744

@Sanchiz Sanchiz merged commit 6ea6e04 into ymcatwincities:8.x-1.x Jun 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR: Code Reviewed Code reviewed. Ready for QA PR: Needs Testing Manual testing is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants