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

ER-1219 Add date published/updated to CDT pages #1268

Closed
wants to merge 15 commits into from

Conversation

martikat
Copy link
Contributor

@martikat martikat commented Jul 17, 2024

ER-1219

This also includes a learning check margin update

Pending changes: Address remaining PR comments and specs are working as expected

@martikat martikat added the review Review app deployed to Azure label Jul 17, 2024
Copy link

@peterdavidhamilton peterdavidhamilton added this to the rc0.15.1 milestone Jul 18, 2024
config/initializers/time_formats.rb Outdated Show resolved Hide resolved
app/views/training/pages/_citation.html.slim Outdated Show resolved Hide resolved
app/models/page.rb Outdated Show resolved Hide resolved
@peterdavidhamilton
Copy link
Contributor

Currently, there is zero test coverage for this. Start with unit testing, first the data objects, then the view layer helpers, and then finally a browser spec if necessary.

@David-Feetenby David-Feetenby added the deployed Review app deployed for testing label Sep 12, 2024
@martikat martikat removed the review Review app deployed to Azure label Sep 12, 2024
@David-Feetenby David-Feetenby added deployed Review app deployed for testing and removed deployed Review app deployed for testing labels Sep 13, 2024
@martikat martikat modified the milestones: rc1.0.1, rc1.0.2 Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployed Review app deployed for testing frontend Changes to assets detected
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants