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 fetching and pushing of currently present data of user in addOrUpdate #2261

Open
6 of 10 tasks
RahulGoyal-tech opened this issue Nov 20, 2024 · 0 comments · May be fixed by #2282
Open
6 of 10 tasks

Remove fetching and pushing of currently present data of user in addOrUpdate #2261

RahulGoyal-tech opened this issue Nov 20, 2024 · 0 comments · May be fixed by #2282
Assignees

Comments

@RahulGoyal-tech
Copy link
Contributor

Issue Description

Currently, In models/users.js, we are fetching current data of user and pushing it while updating profileDiffs, but as { merge : true } is used, this has no effect and no use, hence should be removed.

Expected Behavior

user.data() is not fetched or set, while setting profileDiffs

Current Behavior

user.data() is being fetched and set while updating profileDiffs.

Screenshots

image

Reproducibility

  • This issue is reproducible
  • This issue is not reproducible

Steps to Reproduce

Severity/Priority

  • Critical
  • High
  • Medium
  • Low

Additional Information

Checklist

  • I have read and followed the project's code of conduct.
  • I have searched for similar issues before creating this one.
  • I have provided all the necessary information to understand and reproduce the issue.
  • I am willing to contribute to the resolution of this issue.
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 a pull request may close this issue.

1 participant