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.

Sign up for GitHub

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

Jump to bottom

feat: Remove ParseUI #1783

Merged
merged 1 commit into from
Apr 28, 2024
Merged

feat: Remove ParseUI #1783

merged 1 commit into from
Apr 28, 2024

Conversation

dplewis
Copy link
Member

@dplewis dplewis commented Apr 27, 2024 •
edited
Loading

New Pull Request Checklist

Issue Description

Building a UI has become a lot easier over the years, especially with SwiftUI; ParseUI today looks like an obsolete convenience that could be dropped from the Parse SDK. This is a breaking change.

Closes: #1760
Closes: #1729
Closes: #1493
Closes: #1406
Closes: #1517

Approach

Remove ParseUI.

TODOs before merging

  • Add tests
  • Add changes to documentation (guides, repository pages, in-code descriptions)
  • Remove CI job

@parse-github-assistant Parse GitHub Assistant
Copy link

parse-github-assistant bot commented Apr 27, 2024 •
edited
Loading

Thanks for opening this pull request!

  • 🎉 We are excited about your hands-on contribution!

@dplewis dplewis changed the title Remove ParseUI feat: Remove ParseUI Apr 27, 2024
@dplewis dplewis requested a review from a team April 27, 2024 22:52
@dplewis
Copy link
Member Author

dplewis commented Apr 28, 2024

@mtrezza Codecov is broken

@JohnCaccavale
Copy link
Contributor

Have the Privacy Manifests been added to the project yet? Worth including them here with this PR or best to do that in a separate PR?

@dplewis
Copy link
Member Author

dplewis commented Apr 28, 2024 •
edited
Loading

Have the Privacy Manifests been added to the project yet? Worth including them here with this PR or best to do that in a separate PR?

@JohnCaccavale They haven't been added. If they do it will be separate PR. Can you open an issue for it? I need to do some research on it.

@mtrezza mtrezza merged commit 139eca7 into parse-community:master Apr 28, 2024
8 checks passed
parseplatformorg pushed a commit that referenced this pull request Apr 28, 2024
@semantic-release-bot
chore(release): 4.0.0 [skip ci]
fb1e164 
# [4.0.0](3.0.0...4.0.0) (2024-04-28)

### Features

* Remove `ParseUI` ([#1783](#1783)) ([139eca7](139eca7))

### BREAKING CHANGES

* This release removes `ParseUI`. ([139eca7](139eca7))
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 4.0.0

@parseplatformorg parseplatformorg added the state:released Released as stable version label Apr 28, 2024
@dplewis dplewis deleted the remove-parseui branch May 1, 2024 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JohnCaccavale JohnCaccavale JohnCaccavale approved these changes

@mtrezza mtrezza mtrezza approved these changes

Assignees
No one assigned
Labels
state:breaking state:released Released as stable version
Projects
None yet
Milestone
No milestone
4 participants
@dplewis @JohnCaccavale @parseplatformorg @mtrezza