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

BE | Profile | Military Service History | Fix user reference for academy attendance feature flag #96864

Open
3 tasks
tpharrison opened this issue Nov 12, 2024 · 2 comments
Assignees

Comments

@tpharrison
Copy link
Contributor

Summary

When retrieving military history records for a veteran, there is a conditional check in place that uses the profile_show_military_academy_attendance feature flag to determine whether to display academy attendance records.

The code checks the flag using the @current_user on line in the VAProfile::MilitaryPersonnel::Service. The @current_user is returning nil. It should be changed from @current_user to @user.

Tasks

  • Determine if this code should be removed along with the feature flag.
  • Either remove logic or change @current_user to @user.

Acceptance Criteria

  • Code is removed or fixed.
@ACParker89
Copy link
Contributor

@tpharrison do we need an epic for this or do you see it looped into a past effort?

@tpharrison
Copy link
Contributor Author

@ACParker89 There might be an old epic out there, but I'm not sure what it is. I don't think it needs one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants