Skip to content

Revert "Revert "Revert "Revert "Update student view of Curriculum Catalog cards""""#56583

Merged
TurnerRiley merged 5 commits into
stagingfrom
revert-56582-revert-56517-revert-56507-revert-56322-curriculum-catalog-update-student-view
Feb 16, 2024
Merged

Revert "Revert "Revert "Revert "Update student view of Curriculum Catalog cards""""#56583
TurnerRiley merged 5 commits into
stagingfrom
revert-56582-revert-56517-revert-56507-revert-56322-curriculum-catalog-update-student-view

Conversation

@TurnerRiley

@TurnerRiley TurnerRiley commented Feb 14, 2024

Copy link
Copy Markdown
Contributor

Reverts #56582

Sorry for the re-re-re-revert. I thought the previous change would work because of the passing test following the [test ios] commit, I didn't realize you needed to include [test ios] per commit you wanted tested. This PR does that and I found that the issue was whether "now" was capitalized or not.

As shown below, the show failing baseline [test ios] commit fails, and once the change is implemented it now passes. The reason for the baseline [test ios] commit followed by making the fix then undoing the fix was because I wanted to show a failing commit without the change and a successful one with it, but I pushed both at the same time so drone only ran on the successful one. So, I just undid the fix to get a failing drone build then reimplemented the fix to get a successful one to feel confident that it'll work this time.

@TurnerRiley TurnerRiley requested a review from a team as a code owner February 14, 2024 22:40
@TurnerRiley TurnerRiley requested a review from a team February 15, 2024 20:52

@bethanyaconnor bethanyaconnor left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤞 thanks for the persistence here!

@TurnerRiley TurnerRiley merged commit 1db8fd9 into staging Feb 16, 2024
@TurnerRiley TurnerRiley deleted the revert-56582-revert-56517-revert-56507-revert-56322-curriculum-catalog-update-student-view branch February 16, 2024 18:36
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