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

Fix: Revert to v1.16.9 #1027

Merged
merged 11 commits into from
Dec 19, 2024
Merged

Fix: Revert to v1.16.9 #1027

merged 11 commits into from
Dec 19, 2024

Conversation

VikaCep
Copy link
Contributor

@VikaCep VikaCep commented Dec 19, 2024

In v1.16.10 we introduced an issue where checks can't be saved if the Test button is clicked first.

This happens because the Test button submits the form, but it just doesn't save anything. So the form logic believes that there are no unsaved changes, hence why the handling of "unsaved changes" fails.

Since we're currently on code freeze until Jan 6, 2025, I'm rolling back all changes until the previous working version which is v1.16.9.

@VikaCep VikaCep self-assigned this Dec 19, 2024
@VikaCep VikaCep marked this pull request as ready for review December 19, 2024 19:51
@VikaCep VikaCep requested review from a team as code owners December 19, 2024 19:51
@VikaCep VikaCep requested a review from w1kman December 19, 2024 19:51
@@ -1,3 +1,3 @@
{
".": "1.16.10"
".": "1.16.9"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this cause issues when try to release? (I'm assuming we'll have to release as 1.16.11)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably it would fail, in a following commit I changed the version back to 1.16.10 as it was reverted automatically when running git revert v1.16.9..HEAD

CHANGELOG.md Outdated

* **deps:** bump nanoid from 3.3.7 to 3.3.8 ([#1018](https://github.com/grafana/synthetic-monitoring-app/issues/1018)) ([77032d0](https://github.com/grafana/synthetic-monitoring-app/commit/77032d060323c49ed68394a6e242b0b6aa528d29))
* update checks UI styling ([#1020](https://github.com/grafana/synthetic-monitoring-app/issues/1020)) ([2b69b02](https://github.com/grafana/synthetic-monitoring-app/commit/2b69b024e2759e862b6c1cc490e4490ccf4f4df4))

## [1.16.9](https://github.com/grafana/synthetic-monitoring-app/compare/v1.16.8...v1.16.9) (2024-11-29)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should keep 1.16.10 details if we are releasing as 1.16.11?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. Added it back.

Copy link
Contributor

@peterschretlen peterschretlen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Latest looks good 👍

@VikaCep VikaCep merged commit 51e4d55 into main Dec 19, 2024
5 checks passed
@VikaCep VikaCep deleted the revert-to-v1.16.9 branch December 19, 2024 20:06
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 this pull request may close these issues.

2 participants