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.

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

[Issue 3537] save search get opp #3671

Merged
merged 5 commits into from
Jan 28, 2025
Merged

[Issue 3537] save search get opp #3671

merged 5 commits into from
Jan 28, 2025

Conversation

babebe
Copy link
Collaborator

@babebe babebe commented Jan 28, 2025 •
edited
Loading

Summary

Fixes #{3537}

Time to review: 5 mins

Changes proposed

Added cols last_notified_at and searched_opportunity_ids to UserSavedSearch
Updated factory UserSavedSearchFactory

Screenshot 2025-01-28 at 2 21 35 PM

@babebe babebe linked an issue Jan 28, 2025 that may be closed by this pull request
Create new columns in table user_saved_search #3537
Closed
2 tasks
@babebe babebe merged commit 8ce19d2 into main Jan 28, 2025
@babebe babebe deleted the 3537/save-search-get-opp-id branch January 28, 2025 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chouinar chouinar chouinar approved these changes

@mdragon mdragon Awaiting requested review from mdragon mdragon is a code owner

@acouch acouch Awaiting requested review from acouch acouch is a code owner

@andycochran andycochran Awaiting requested review from andycochran andycochran is a code owner

@btabaska btabaska Awaiting requested review from btabaska btabaska is a code owner

@coilysiren coilysiren Awaiting requested review from coilysiren coilysiren is a code owner

@doug-s-nava doug-s-nava Awaiting requested review from doug-s-nava doug-s-nava is a code owner

@widal001 widal001 Awaiting requested review from widal001 widal001 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create new columns in table user_saved_search
3 participants
@babebe @chouinar @nava-platform-bot