Skip to content

[PROD] #3215

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.

Sign up for GitHub

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

Jump to bottom
Merged
elainaparrish merged 5 commits into from
Jan 6, 2026
Merged

[PROD] #3215

elainaparrish merged 5 commits into from
Jan 6, 2026

Conversation

@kryswisnaskas
Copy link
Collaborator

@kryswisnaskas kryswisnaskas commented Jan 6, 2026 •
edited
Loading

Description of change

TTAHUB-4720 - Allow approver edits after approval
NO-TICKET - Add missing success messages to session report/TR landing
NO-TICKET - Fix missing deps in env reset cron
TTAHUB-4671 - Address 3 dependabot alerts

How to test

Issue(s)

Checklists

Every PR

  • Meets issue criteria
  • JIRA ticket status updated
  • Code is meaningfully tested

Production Deploy

  • PR created as Draft
  • Staging smoke test completed
  • PR transitioned to Open
  • Reviewer added (after transitioning to Open to ensure Slack notifications trigger)
    • Sequence: Draft PR → Smoke test → Open PR → Add reviewer
    • Confirm that Slack notification was sent after reviewer was added

After merge/deploy

  • Update JIRA ticket status

dependabot bot and others added 5 commits January 2, 2026 18:12
@dependabot @kryswisnaskas
Bump qs from 6.14.0 to 6.14.1 in /frontend (#3208)
a79e757 
* Bump qs from 6.14.0 to 6.14.1 in /frontend

Bumps [qs](https://github.com/ljharb/qs) from 6.14.0 to 6.14.1.
- [Changelog](https://github.com/ljharb/qs/blob/main/CHANGELOG.md)
- [Commits](ljharb/qs@v6.14.0...v6.14.1)

---
updated-dependencies:
- dependency-name: qs
  dependency-version: 6.14.1
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

* Clean up merge conflict markers

* Modify resolutions to include the updated qs version

* Fix tests (2026 bug)

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Krys Wisnaskas <[email protected]>
@tommaroh
[NO-TICKET] Fix missing deps in env reset cron (#3212)
20e51d6 
* install deps before deploy

* testing

* ordering

* uncomment testing lines
@thewatermethod
Add alert and success messages (#3213)
b465e25
@kryswisnaskas
[TTAHUB-4720] Allow approver edits after approval (#3209)
bae2cde 
* Allow approver edits after approval and re-entry

* Update failing tests

* Add frontend tests

* Remove stray files

* Improve test coverage
@kryswisnaskas
Update fengari (#3211)
136c711
@kryswisnaskas kryswisnaskas requested review from elainaparrish and removed request for elainaparrish January 6, 2026 14:52
@kryswisnaskas kryswisnaskas marked this pull request as ready for review January 6, 2026 14:53
@elainaparrish elainaparrish merged commit 7dd8406 into production Jan 6, 2026
44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@elainaparrish elainaparrish elainaparrish approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@kryswisnaskas @elainaparrish @tommaroh @thewatermethod