Skip to content
This repository has been archived by the owner on Jun 1, 2024. It is now read-only.

fix: make unowned references weak #1443

Merged
merged 1 commit into from
Mar 23, 2021
Merged

fix: make unowned references weak #1443

merged 1 commit into from
Mar 23, 2021

Conversation

lucasaraujo
Copy link
Contributor

Description and Example

Remove unowned references

Checklist

  • I read the PR Guide and followed the process outlined there for submitting this PR.
  • I avoided breaking changes by not changing public APIs that people rely on.
  • I am willing to follow-up on review comments in a timely manner.

@lucasaraujo lucasaraujo added the iOS This issue directly affects structure and logic of iOS project label Mar 17, 2021
@botbeaglezup
Copy link
Contributor

botbeaglezup commented Mar 17, 2021

2 Warnings
⚠️ iOS/Sources/Beagle/Sources/Analytics/Deprecated/DeprecatedAnalytics.swift#L19 - TODOs should be resolved (Colocar deprecated).
todo DeprecatedAnalytics.swift:19
⚠️ iOS/Sources/Beagle/Sources/Renderer/BeagleRenderer.swift#L135 - TODOs should be resolved (should we make observeMany t…).
todo BeagleRenderer.swift:135

Generated by 🚫 Danger

@lucasaraujo lucasaraujo force-pushed the ios/unowned-to-weak branch from 34837b0 to 7aa8455 Compare March 17, 2021 16:44
Copy link
Contributor

@theffc theffc left a comment

Choose a reason for hiding this comment

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

  • suggested changes on some places that were using dependencies through renderer.controller?.dependencies instead of using the new renderer.dependencies
  • some other comments

@lucasaraujo lucasaraujo force-pushed the ios/unowned-to-weak branch from 7aa8455 to 8f08420 Compare March 18, 2021 19:44
@lucasaraujo lucasaraujo force-pushed the ios/unowned-to-weak branch from 8f08420 to cfc11d2 Compare March 18, 2021 19:56
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@dantes-git dantes-git requested a review from theffc March 22, 2021 18:38
@dantes-git dantes-git merged commit c6214bf into master Mar 23, 2021
@dantes-git dantes-git deleted the ios/unowned-to-weak branch March 23, 2021 15:01
dantes-git pushed a commit that referenced this pull request Mar 23, 2021
BREAKING CHANGE: the property controller in BeagleRenderer is now weak
uziasferreirazup pushed a commit that referenced this pull request Mar 24, 2021
uziasferreirazup added a commit that referenced this pull request Mar 29, 2021
* test components

* feat: improve api to write flex and style

* adjust code

* remove components

* add new components

* create extension to unit value

* adjust backend code and create sized box component

* add new function in style

* adjust operation

* create static function in bind

* improve accessbility

* create helper to platform

* apply var to corner radius

* adjust code

* add simple documentation

* adjust documentation

* adjust code

* create unit test

* create unit test

* adjust code

* create unit test

* adjust test and components

* adjust expanded

* fix: PageView retain cycle (#1434)

* Fix PageView retain cycle

* PageView retain cycle test

* chore: adjust error in horusec (#1441)

* chore: ignore error in appium test

* adjust

* adjust

* fix page layout of listView tests (#1423)

* feat: Improve iOS test script setup step (#1429)

* update

* update

* update

* update

* update

* update

* update

* update

* update

* update

* update

* update

* update

* update

* update

* update

* Update automated_tests_appium_ios.yml

update

* Update backend/framework/src/main/kotlin/br/com/zup/beagle/platform/BeaglePlatform.kt

Co-authored-by: igorgiumellizup <[email protected]>

* Update backend/kotlin-core/src/main/kotlin/br/com/zup/beagle/ext/FlexExtension.kt

Co-authored-by: igorgiumellizup <[email protected]>

* Update backend/kotlin-core/src/main/kotlin/br/com/zup/beagle/ext/FlexExtension.kt

Co-authored-by: igorgiumellizup <[email protected]>

* Update backend/kotlin-core/src/main/kotlin/br/com/zup/beagle/ext/FlexExtension.kt

Co-authored-by: igorgiumellizup <[email protected]>

* Update backend/kotlin-core/src/main/kotlin/br/com/zup/beagle/ext/StyleExtension.kt

Co-authored-by: igorgiumellizup <[email protected]>

* Update backend/kotlin-core/src/main/kotlin/br/com/zup/beagle/ext/StyleExtension.kt

Co-authored-by: igorgiumellizup <[email protected]>

* Update backend/widgets/src/main/kotlin/br/com/zup/beagle/widget/style/Margin.kt

Co-authored-by: igorgiumellizup <[email protected]>

* Update backend/widgets/src/main/kotlin/br/com/zup/beagle/widget/style/Padding.kt

Co-authored-by: igorgiumellizup <[email protected]>

* Update backend/widgets/src/main/kotlin/br/com/zup/beagle/widget/style/SizedBox.kt

Co-authored-by: igorgiumellizup <[email protected]>

* Update backend/widgets/src/main/kotlin/br/com/zup/beagle/widget/style/SizedBox.kt

Co-authored-by: igorgiumellizup <[email protected]>

* Update backend/widgets/src/main/kotlin/br/com/zup/beagle/widget/style/SizedBox.kt

Co-authored-by: igorgiumellizup <[email protected]>

* fix: iOS - TextInput enabled custom style config (#1440)

* fix: iOS - ErrorView retry block (#1428)

* fix: adjust crash when passing color in page indicator (#1452)

* tests removal (#1448)

* make unowned references weak (#1443)

* Update backend/framework/src/main/kotlin/br/com/zup/beagle/platform/BeaglePlatform.kt

Co-authored-by: Matheus Ribeiro Lima <[email protected]>

* Update backend/framework/src/main/kotlin/br/com/zup/beagle/platform/BeaglePlatform.kt

Co-authored-by: Matheus Ribeiro Lima <[email protected]>

* Update backend/widgets/src/main/kotlin/br/com/zup/beagle/widget/style/Accessibility.kt

Co-authored-by: Matheus Ribeiro Lima <[email protected]>

* adjust style

* adjust corner radius in platform

* remove builders

* adjust set flex

* adjust name flexible

* add new attribute has in master

* Update backend/kotlin-core/src/main/kotlin/br/com/zup/beagle/ext/FlexExtension.kt

Co-authored-by: paulomeurerzup <[email protected]>

* Update backend/kotlin-core/src/main/kotlin/br/com/zup/beagle/ext/FlexExtension.kt

Co-authored-by: paulomeurerzup <[email protected]>

* adjust snapshot test

Co-authored-by: Lucas Araújo <[email protected]>
Co-authored-by: tayronemachadozup <[email protected]>
Co-authored-by: igorgiumellizup <[email protected]>
Co-authored-by: Daniel Tes <[email protected]>
Co-authored-by: Matheus Ribeiro Lima <[email protected]>
Co-authored-by: paulomeurerzup <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
iOS This issue directly affects structure and logic of iOS project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants