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

Track custom field changes for correction #7023

Merged
merged 4 commits into from
Jul 4, 2024
Merged

Track custom field changes for correction #7023

merged 4 commits into from
Jul 4, 2024

Conversation

Nil20
Copy link
Collaborator

@Nil20 Nil20 commented May 21, 2024

No description provided.

@Nil20
Copy link
Collaborator Author

Nil20 commented May 21, 2024

I've kept the Burkina Faso's temporary patch as the current fix as well. I don't think we can do some fix which could be marginally better(Doesn't mean it won't be another patch) and doesn't break something else in the process.
The PR's open for brainstorming ideas about it anyway.

@Zangetsu101
Copy link
Collaborator

Zangetsu101 commented Jun 3, 2024

Riku has created some dev tasks for this ticket here: #6810 so I'm closing this ticket for now. @Nil20 you can reopen it when you get back to it or if someone else picks it up

@Zangetsu101 Zangetsu101 closed this Jun 3, 2024
@rikukissa
Copy link
Member

@Zangetsu101 Tahmid and Markus will need this in Madagascar so we need to get it to 1.5.0 😬

@Zangetsu101
Copy link
Collaborator

@Nil20 Let's update CHANGELOG please and then we can merge it in

@Zangetsu101 Zangetsu101 added this to the v1.5.0 milestone Jul 4, 2024
@Zangetsu101 Zangetsu101 merged commit 35f993a into develop Jul 4, 2024
22 checks passed
github-actions bot pushed a commit that referenced this pull request Jul 4, 2024
Nil20 added a commit that referenced this pull request Jul 4, 2024
* chore: rename constants related to fieldChange functions

* fix: track custom field value changes
Zangetsu101 added a commit that referenced this pull request Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants