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: docshare on int pk doctype #19832

Merged
merged 1 commit into from
Jan 30, 2023
Merged

fix: docshare on int pk doctype #19832

merged 1 commit into from
Jan 30, 2023

Conversation

ankush
Copy link
Member

@ankush ankush commented Jan 30, 2023

DocShare doesn't work on doctypes which have int PKs, this PR fixes that.

@ankush ankush requested review from surajshetty3416 and a team as code owners January 30, 2023 06:55
@ankush ankush requested review from phot0n and removed request for a team January 30, 2023 06:55
@ankush ankush changed the title fix: Convert doctype name to string fix: docshare on int pk doctype Jan 30, 2023
@ankush ankush added the backport version-14-hotfix backport to version 14 label Jan 30, 2023
@ankush ankush added the Skip CI Doesn't run Ci for this PR. label Jan 30, 2023
@ankush ankush merged commit 1eab4e4 into frappe:develop Jan 30, 2023
@ankush ankush deleted the shared_int_pk branch January 30, 2023 08:41
mergify bot pushed a commit that referenced this pull request Jan 30, 2023
ankush added a commit that referenced this pull request Jan 30, 2023
(cherry picked from commit 1eab4e4)

Co-authored-by: Ankush Menat <ankush@frappe.io>
frappe-pr-bot pushed a commit that referenced this pull request Jan 30, 2023
# [14.25.0](v14.24.0...v14.25.0) (2023-01-30)

### Bug Fixes

* add freeze message for bulk delete ([2a42036](2a42036))
* assertAlmostEqual with precision ([#19794](#19794)) ([9f7c4e0](9f7c4e0))
* Convert doctype name to string ([#19832](#19832)) ([#19834](#19834)) ([a45f31d](a45f31d))
* correct exit code on missing app failure ([#19676](#19676)) ([#19770](#19770)) ([f6139a4](f6139a4))
* **i18n:** Datepicker Turkish translations ([#19777](#19777)) ([#19831](#19831)) ([3e91fb1](3e91fb1))
* incorrect link when std field has problem (backport [#19744](#19744)) ([#19763](#19763)) ([4593bb9](4593bb9))
* **MariaDBTable:** dont attempt to drop index twice ([#19783](#19783)) ([67f80c6](67f80c6))
* Password strength check for long passwords (backport [#19756](#19756)) ([#19765](#19765)) ([a6315f9](a6315f9))
* respect disable sidebar stats on list view ([#19795](#19795)) ([5f57816](5f57816))
* sanitize traceback for common secrets ([#19805](#19805)) ([#19806](#19806)) ([ae6f2b1](ae6f2b1))
* use count instead of concatenated docnames ([06948d1](06948d1))

### Features

* Audit hooks report (backport [#19780](#19780)) ([#19828](#19828)) ([99bdf34](99bdf34))
* better freeze message ([c03f9e7](c03f9e7))
stephenBDT pushed a commit to alias/frappe that referenced this pull request Feb 7, 2023
(cherry picked from commit 1eab4e4)

Co-authored-by: Ankush Menat <ankush@frappe.io>
stephenBDT pushed a commit to alias/frappe that referenced this pull request Feb 7, 2023
# [14.25.0](frappe/frappe@v14.24.0...v14.25.0) (2023-01-30)

### Bug Fixes

* add freeze message for bulk delete ([2a42036](frappe@2a42036))
* assertAlmostEqual with precision ([frappe#19794](frappe#19794)) ([9f7c4e0](frappe@9f7c4e0))
* Convert doctype name to string ([frappe#19832](frappe#19832)) ([frappe#19834](frappe#19834)) ([a45f31d](frappe@a45f31d))
* correct exit code on missing app failure ([frappe#19676](frappe#19676)) ([frappe#19770](frappe#19770)) ([f6139a4](frappe@f6139a4))
* **i18n:** Datepicker Turkish translations ([frappe#19777](frappe#19777)) ([frappe#19831](frappe#19831)) ([3e91fb1](frappe@3e91fb1))
* incorrect link when std field has problem (backport [frappe#19744](frappe#19744)) ([frappe#19763](frappe#19763)) ([4593bb9](frappe@4593bb9))
* **MariaDBTable:** dont attempt to drop index twice ([frappe#19783](frappe#19783)) ([67f80c6](frappe@67f80c6))
* Password strength check for long passwords (backport [frappe#19756](frappe#19756)) ([frappe#19765](frappe#19765)) ([a6315f9](frappe@a6315f9))
* respect disable sidebar stats on list view ([frappe#19795](frappe#19795)) ([5f57816](frappe@5f57816))
* sanitize traceback for common secrets ([frappe#19805](frappe#19805)) ([frappe#19806](frappe#19806)) ([ae6f2b1](frappe@ae6f2b1))
* use count instead of concatenated docnames ([06948d1](frappe@06948d1))

### Features

* Audit hooks report (backport [frappe#19780](frappe#19780)) ([frappe#19828](frappe#19828)) ([99bdf34](frappe@99bdf34))
* better freeze message ([c03f9e7](frappe@c03f9e7))
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport version-14-hotfix backport to version 14 Skip CI Doesn't run Ci for this PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant