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

feat(apcd-cms): core-cms v3.11.0 candidate #159

Merged
merged 11 commits into from
May 25, 2023
Merged

Conversation

wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented May 20, 2023

Status

Overview

Change APCD CMS to rely on upcoming Core-CMS v3.11 code instead of TACC/Core-CMS#567.*

* TACC/Core-CMS#567 was v3.9 + 25 commits of v3.10.

  1. Finally brings belated Core-Styles to APCD.
  2. Tests v3.11 on the CMS most cumbersome to upgrade.
  3. Removes cruft and duplicate CSS.

Related

Changes

UI

  • new table styles
  • fixed link hover state (border was dashed and too close to text)
  • new accent color (purple, matches portal)

Code

  • changed core cms image
  • added load of new apcd table and modal stylesheets
  • removed load of unused form stylesheets (now available from core)
  • removed stylesheets, or code from stylesheets, whose code is now in apcd table and modal stylesheets
    • per-page css is retained in per-page stylesheets
  • changed pagination to load from core-styles v0.13 (instead of v0.10)
  • added conflicts-with-core css files to deal with apcd still being both a bit ahead and behind core
  • removed most table desktop styles

Testing

Deployed to https://apcd-qa.tacc.utexas.edu/ on 2023-05-21.

  1. Compare all pages to pre-prod before deploy of new CMS version.

UI

Register
before after
Request to Submit register_request-to-submit_ register_request-to-submit_
Submissions
before after
List Submissions submissions_list-submissions_ submissions_list-submissions_
Request Exception submissions_exception_ submissions_exception_
Request Exception Form OLD submissions_threshold-exception_ NEW submissions_threshold-exception_
Request Extension submissions_extension-request_ submissions_extension-request_
Request Extension Form OLD submissions_extension-request_ NEW submissions_extension-request_
Administration
before after
List Submissions administration_list-submissions_ administration_list-submissions_
List Extension Requests administration_list-extensions_ administration_list-extensions_
List Exception Requests administration_list-exceptions_ administration_list-exceptions_
List Users administration_view-users administration_view-users_
List Registrations administration_list-registration-requests_ administration_list-registration-requests_
Help
before after
Reference help_reference_ help_reference_

Notes

Known Issues

  1. https://apcd-qa.tacc.utexas.edu/submissions/exception/ buttons do not have hover state.
    • Pre-existing bug.
    • Caused by APCD markup not using c-button c-button--primary.
  2. Font that is regular weight and italic style is too thick.
    • Pre-existing bug which now occurs in modal forms (because styling is now consistent).
    • See TACC/Core-CMS#638 "Known Issues" № 6.
  3. Accent color has changed from blue to purple.
  4. Pagination styles are wrong on Review Extensions and Review Exceptions pages.
    • Pre-existing bug.
    • Reported in WP-150.

@wesleyboar wesleyboar changed the title test(apcd): core-cms v3.11.0 test(apcd-cms): core-cms v3.11.0 May 20, 2023
@wesleyboar wesleyboar changed the title test(apcd-cms): core-cms v3.11.0 feat(apcd-cms): core-cms v3.11.0 May 22, 2023
@wesleyboar wesleyboar changed the title feat(apcd-cms): core-cms v3.11.0 feat(apcd-cms): core-cms v3.11.0 candidate May 22, 2023
@wesleyboar wesleyboar marked this pull request as ready for review May 22, 2023 17:26
@wesleyboar wesleyboar requested review from edmondsgarrett, nathanfranklin and fnets and removed request for nathanfranklin May 23, 2023 17:39
Copy link
Contributor

@edmondsgarrett edmondsgarrett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Things are looking great on QA! Approved.

@wesleyboar
Copy link
Member Author

wesleyboar commented May 23, 2023

Cool!

I found some things I may do before I merge:

Footnotes

  1. Requires I be in the database. 2

@wesleyboar
Copy link
Member Author

@wesleyboar wesleyboar merged commit a088b69 into main May 25, 2023
@wesleyboar wesleyboar deleted the task/v3.11.0-candidate branch May 25, 2023 23:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants