Skip to content

Fix empty _rev parameter from dropdown select list#1482

Open
big-r81 wants to merge 1 commit into
mainfrom
fix-empty-rev-param
Open

Fix empty _rev parameter from dropdown select list#1482
big-r81 wants to merge 1 commit into
mainfrom
fix-empty-rev-param

Conversation

@big-r81

@big-r81 big-r81 commented Jun 27, 2025

Copy link
Copy Markdown
Contributor

If there are multiple conflict versions in the dropdown field, the http get request was called with an empty rev parameter and failed. Use the right child object to get the value.

Example:

Before:
GET /b/1?rev= HTTP/1.1

After:
GET /b/1?rev=1-03a8cbc32975d2f58295a1ea2617d965 HTTP/1.1

@big-r81
big-r81 requested a review from Antonio-Maranhao June 30, 2025 21:22
If there are multiple conflict versions in the dropdown field,
the http get request was called with an empty rev parameter and failed.
Use the right child object to get the value.
@big-r81
big-r81 force-pushed the fix-empty-rev-param branch from 6e5f2c7 to 5376b46 Compare August 7, 2025 12:30
@big-r81

big-r81 commented Dec 9, 2025

Copy link
Copy Markdown
Contributor Author

@janl friendly nudging. The question is, if target.value is the correct child object for this. It seems to work ...

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.

1 participant