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

Remove start-up event and revue collab from constants #3716

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

viljeno
Copy link

@viljeno viljeno commented Jan 22, 2025

Updated interest form as requested by Intercom.

ABA-1232

@viljeno viljeno added the review-needed Pull requests that need review label Jan 22, 2025
@viljeno viljeno requested review from falbru and Arashfa0301 January 22, 2025 19:38
@viljeno viljeno self-assigned this Jan 22, 2025
@viljeno viljeno force-pushed the update-interest-form branch from 7db87e8 to 9119c7e Compare January 22, 2025 21:32
Copy link

codecov bot commented Jan 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.67%. Comparing base (41d8412) to head (9119c7e).
Report is 6 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3716   +/-   ##
=======================================
  Coverage   89.67%   89.67%           
=======================================
  Files         703      704    +1     
  Lines       22534    22537    +3     
=======================================
+ Hits        20207    20210    +3     
  Misses       2327     2327           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ivarnakken
Copy link
Member

We probably want to use these fields later, so we probably shouldn't remove this from the backend model. This would actually remove these values from previous interest forms. Code looks good, but I suggest closing this PR since it's not something we should remove.

@ivarnakken ivarnakken added the do-not-merge/hold Pull request is on hold (e.g. waiting for something else to be merged), and should not be merged label Jan 29, 2025
@ivarnakken
Copy link
Member

Just to clarify: updating the frontend was enough! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/hold Pull request is on hold (e.g. waiting for something else to be merged), and should not be merged review-needed Pull requests that need review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants