Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

SWSDEVOPS 150 - trustee profile update #76

Open
wants to merge 72 commits into
base: 5.x
Choose a base branch
from
Open

SWSDEVOPS 150 - trustee profile update #76

wants to merge 72 commits into from

Conversation

mdyoung3
Copy link
Collaborator

@mdyoung3 mdyoung3 commented Feb 4, 2025

NOT READY FOR REVIEW

  • (Edit the above to reflect status)

Summary

  • TL;DR - what's this PR for?

Review By (Date)

  • When does this need to be reviewed by?

Criticality

  • How critical is this PR on a 1-10 scale? Also see Severity Assessment.
  • E.g., it affects one site, or every site and product?

Urgency

  • How urgent is this? (Normal, High)

Review Tasks

Setup tasks and/or behavior to test

  1. Check out this branch
  2. Rebuild Cache and import config drush cr ; drush ci
  3. Navigate to...
  4. Verify...

Site Configuration Sync

  • Is there a config:export in this PR that changes the config sync directory?

Front End Validation

Backend / Functional Validation

Code

  • Are the naming conventions following our standards?
  • Does the code have sufficient inline comments?
  • Is there anything in this code that would be hidden or hard to discover through the UI?
  • Are there any code smells?
  • Are tests provided? eg (unit, behat, or codeception)

Code security

General

  • Is there anything included in this PR that is not related to the problem it is trying to solve?
  • Is the approach to the problem appropriate?

Affected Projects or Products

  • Does this PR impact any particular projects, products, or modules?

Associated Issues and/or People

  • JIRA ticket(s)
  • Other PRs
  • Any other contextual information that might be helpful (e.g., description of a bug that this PR fixes, new functionality that it adds, etc.)
  • Anyone who should be notified? (@mention them here)

Resources

buttonwillowsix and others added 30 commits September 30, 2024 22:02
@buttonwillowsix
D8Core-7515: adding a few additional options (#819)
e04bd22
@pookmish
Removed unused page_cache_query_ignore module
0a0f8ca
@rebeccahongsf
D8CORE-7517 | update event subtitle heading (#816)
17abf11
@buttonwillowsix
D8CORE-7564: removed decanter link and replaced with identity guide (…
6e85c3f 
…#822)

* D8CORE-7564: removed decanter link and replaced with identity guide info.

* D8CORE-7564: added in a handy link
@pookmish
Remove edit links in views (#821)
9cd43c5
@pookmish
11.4.5
5fec035
@pookmish
Added graphql view for search results
647aad4
@pookmish
Fix acceptance tests after contrib updates (#824)
af190d0
@pookmish
Update layout builder modal module
9c3b933
@pookmish
Added modal colorbox image display choices in the wysiwyg (#826)
ca9e29e
@pookmish
Added FAQ Accordion List paragraph type based on CAW (#827)
e1d68a3
@jenbreese
D8CORE-7558: added the global footer to minimal theme in dark with no…
7939306 
… logo (#820)
@pookmish
D8CORE-7578 Adjust person display for lists and improve resolution (#…
c3a7ef6 
…828)
@rebeccahongsf
D8CORE-7434 D8CORE-7435 D8CORE-7436 | Opportunity full content layout…
9d7fa73 
… and configure card and list display (#813)
@pookmish
D8CORE-7454: Move back-to-top inside the <main> element
44a9c9f
@pookmish
D8CORE-6821 Adjust skip to main content location
e3755a1
@pookmish
Added aria label to masthead section
8a596db
@pookmish
D8CORE-5828 If home page starts with a wysiwyg, add some space at the…
58b7f31 
… top
@pookmish
D8CORE-6644 Omit default favicons if one is uploaded in the theme set…
a3b380e 
…tings
@pookmish
D8CORE-7194 Embed codes with captions respect 100% widths on iframes
94fddf9
@pookmish
Exposed opportunity in graphql
7ab757b
@pookmish
Updated metatag module config
c6d4495
@pookmish
Updated metadata image sizes
1d0e25b
@pookmish
Adjusted person term page list
a0610a1
@pookmish
Restrict accordion answer body and fix contact information redirect l…
a32efc1 
…ogic
@pookmish
Fixed unit test
a634495
@pookmish
Fix stanford basic yarn install
fac5713
@pookmish
11.5.0
451f604
@pookmish
Updated config from db update
9d69171
@pookmish
Update ultimate cron config after module update (#832)
c2be7e9
jenbreese and others added 16 commits January 8, 2025 13:15
@jenbreese
D8CORE-7532: removed wysiwyg p line length (#842)
94b55fb
@buttonwillowsix
D8CORE-7703: fixed bad link, updated text (#844)
9dd66f7
@jenbreese
D8CORE-7531: removed max-width on intro text and drop cap text (#843)
e530414
@pookmish
Update tests to work correctly with trash module (#850)
6dd9de5
@pookmish
Revert "Add and configure trash module for soft deleteing content (#8…
2aba8bd 
…35)" (#851)

This reverts commit 9337155.
@buttonwillowsix
D8Core-7339: added help text and additional icons to social links in …
7deb42a 
…footer. (#846)
@jenbreese
D8CORE-7220: h1 on Homepage (#847)
ddf40bd 
* D8CORE-7220: h1 on hp

* fixup

* removing extra lines

* D8CORE-7220: h1 on homepage only

* Fixup to class name
@buttonwillowsix
D8CORE-7644: updated user creation mail (#856)
1626eee
@jenbreese
D8CORE-7339: added net to the local footer social link validation (#859)
cae8bcd
@buttonwillowsix
D8CORE-7622: Site Reviewer Role (#855)
25efbd6
@buttonwillowsix
D8CORE-7780: Content Management pages for person, event and news (#858)
ff4fa32
@buttonwillowsix
D8CORE-7705: Self-Service Enhanced Search (#861)
1c8b713
@buttonwillowsix
Fix typo in lockup config page settings. (#862)
1947be9 
Fixed spelling error
@pookmish
D8CORE 7753 refactor accordion paragraph to use button and div elemen…
8259872 
…ts (#863)

* D8CORE-7753: Refactor accordions for buttons and divs

* dev version profile helper

* updated test

* Revert "dev version profile helper"

This reverts commit 2f39942f9918a6d95cc179855b4d4737dddb552e.
@pookmish
11.5.1
007c455
@pookmish
11.5.2
c7723b0 
- Removed non-existent permissions
buttonwillowsix and others added 9 commits February 10, 2025 11:02
@buttonwillowsix
D8CORE-7806: Gave site owners permissions to assign Reviewer role. (#…
b503955 
…868)
@pookmish
11.5.3
80ef89c
@jenbreese
D8CORE-7813: fixed the text color in the more publications button (#870)
4c57659
@mdyoung3
Merge branch '11.x' of https://github.com/SU-SWS/stanford_profile int…
b59c8d1 
…o SWSDEVOPS-150-trustee-profile-update
@jenbreese
D8CORE-7663: reorganized the opportunity node page (#833)
9a33956 
* D8CORE-7663: reorged the opportunity node page

* D8CORE-7663: fixup to layout

* D8CORE-7663:moved around elements

* Made email required and moved around content and added classes
@mdyoung3
Merge branch '11.x' of https://github.com/SU-SWS/stanford_profile int…
0ba36d8 
…o SWSDEVOPS-150-trustee-profile-update
@mdyoung3
Updated dependencies and enable allowed formats
1959443
@mdyoung3
Added upgrade status
2af676f
@mdyoung3
Minor version on next module 1.2 -> 1.6
e2c06ee
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
patch size/xl
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@mdyoung3 @buttonwillowsix @pookmish @rebeccahongsf @jenbreese