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: ignore internal methods #18784

Merged
merged 1 commit into from Nov 6, 2022
Merged

fix: ignore internal methods #18784

merged 1 commit into from Nov 6, 2022

Conversation

ankush
Copy link
Member

@ankush ankush commented Nov 6, 2022

@ankush ankush requested a review from a team as a code owner November 6, 2022 13:47
@ankush ankush requested review from shariquerik and removed request for a team November 6, 2022 13:47
@github-actions github-actions bot added the add-test-cases Add test case to validate fix or enhancement label Nov 6, 2022
@ankush ankush removed the add-test-cases Add test case to validate fix or enhancement label Nov 6, 2022
@ankush ankush merged commit 44a5bdc into frappe:develop Nov 6, 2022
@ankush ankush deleted the flatten_fix branch November 6, 2022 13:48
ankush added a commit that referenced this pull request Nov 8, 2022
ankush added a commit that referenced this pull request Nov 8, 2022
#18806)

* fix: dont allow reading attributes of unsafe objects (#18706)

(cherry picked from commit e02b90c)

* fix: ignore internal methods (#18784)

Co-authored-by: Ankush Menat <ankush@frappe.io>
frappe-pr-bot pushed a commit that referenced this pull request Nov 15, 2022
# [14.15.0](v14.14.3...v14.15.0) (2022-11-15)

### Bug Fixes

* avoid patching QB if already patched ([a8ba877](a8ba877))
* breadcrumbs is broken ([90ca7b1](90ca7b1))
* check if the doctype exists before adding default logtypes in log settings ([#18867](#18867)) ([#18869](#18869)) ([93fe3e9](93fe3e9))
* dashboard view from workspace (backport [#18779](#18779)) ([#18813](#18813)) ([42f5e04](42f5e04))
* decorator ordering ([2245cc0](2245cc0))
* dont allow reading attributes of unsafe objects (backport [#18706](#18706)) ([#18806](#18806)) ([2b193bd](2b193bd)), closes [#18784](#18784)
* hardcode doctype in google oauth callback ([#18862](#18862)) ([78a337b](78a337b))
* page has an empty menu button ([b2bbcde](b2bbcde))
* raise exception if doc before save is not found ([#18796](#18796)) ([#18820](#18820)) ([7505a0d](7505a0d))
* reportview permlevel bug ([#18822](#18822)) ([#18828](#18828)) ([ec248a9](ec248a9))
* **security:** prevent xss attack in search ([#18847](#18847)) ([#18850](#18850)) ([8807035](8807035))

### Features

* Set default SQL statement timeouts ([8ab95f4](8ab95f4))
* show utilization percent on RQ Worker ([#18868](#18868)) ([#18870](#18870)) ([a7060fa](a7060fa))

### Performance Improvements

* faster as_list and as_dict ([33d6369](33d6369))
* faster generate_hash (backport [#18825](#18825)) ([#18851](#18851)) ([bb5c7d5](bb5c7d5))
* **workflow:** get_transitions ([#18834](#18834)) ([#18839](#18839)) ([3f71181](3f71181))

### Reverts

* async await in breadcrumbs.js ([6a1e619](6a1e619))
ankush pushed a commit that referenced this pull request Nov 15, 2022
#18806)

* fix: dont allow reading attributes of unsafe objects (#18706)

(cherry picked from commit e02b90c)

* fix: ignore internal methods (#18784)

Co-authored-by: Ankush Menat <ankush@frappe.io>
(cherry picked from commit 2b193bd)

fix: avoid patching QB if already patched
ankush pushed a commit that referenced this pull request Nov 15, 2022
#18806) (#18884)

* fix: dont allow reading attributes of unsafe objects (#18706)

(cherry picked from commit e02b90c)

* fix: ignore internal methods (#18784)

Co-authored-by: Ankush Menat <ankush@frappe.io>
(cherry picked from commit 2b193bd)

fix: avoid patching QB if already patched

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant