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

Reply support #98

Merged
merged 20 commits into from
Aug 11, 2022
Merged

Reply support #98

merged 20 commits into from
Aug 11, 2022

Conversation

cryptoAlgorithm
Copy link
Member

Now, you can reply to messages right from Swiftcord!

This is obviously not how a ViewModel is used, but this commit was growing a little too big for comfort.
Supplying width and height arguments are no longer required
Instead of switching between a CachedImageView and SwiftyGifView
Run blocking operations on the utility DispatchQueue to prevent hanging the main thread
* Use BetterImageView where possible to increase conciseness
* Remove animation from server button when pressed (like Discord)
* Make channel icon in message list header align with the official client
Newly supported types:
* recipientAdd (member added to a group DM)
* recipientRemove (member removed from group DM)
* userPremiumGuildSub (server boosted)
* userPremiumGuildSubTier1, userPremiumGuildSubTier2, userPremiumGuildSubTier3 (server boosted and achieved tiers 1, 2 or 3 respectively)
Note: Every commit from now onwards expects that Message is a class and referenced_message is of type Message.
commit c54920f
Author: cxt <[email protected]>
Date:   Mon Aug 8 18:23:17 2022 +0000

    Translated using Weblate (German)

    Currently translated at 98.1% (106 of 108 strings)

    Translation: Swiftcord/Swiftcord
    Translate-URL: https://hosted.weblate.org/projects/swiftcord/swiftcord/de/

commit 81cde3d
Author: jnghyk <[email protected]>
Date:   Mon Aug 8 11:37:50 2022 +0000

    Translated using Weblate (Korean)

    Currently translated at 100.0% (82 of 82 strings)

    Translation: Swiftcord/Swiftcord
    Translate-URL: https://hosted.weblate.org/projects/swiftcord/swiftcord/ko/

commit dfc2c03
Author: jnghyk <[email protected]>
Date:   Mon Aug 8 11:15:47 2022 +0000

    Translated using Weblate (Korean)

    Currently translated at 98.7% (81 of 82 strings)

    Translation: Swiftcord/Swiftcord
    Translate-URL: https://hosted.weblate.org/projects/swiftcord/swiftcord/ko/

commit 8782877
Author: vinkwok <[email protected]>
Date:   Thu Aug 4 22:24:03 2022 +0800

    Initial reply UI

commit 0b2fc65
Author: vinkwok <[email protected]>
Date:   Wed Aug 10 18:38:36 2022 +0800

    Optimisation: play/pause animation when required

    Instead of switching between a CachedImageView and SwiftyGifView
Self cannot be mutated since mutable functions don’t exist in classes, so return a new instance instead. This should fix this actions build failure.
In progress of debugging some GitHub actions issue. Also includes minor pixel-perfection improvements.
In progress of debugging some GitHub actions issue. Also includes minor pixel-perfection improvements.
No longer uses the flaky MessageInfoBarView method
@cryptoAlgorithm cryptoAlgorithm marked this pull request as ready for review August 11, 2022 12:37
commit f8b4a24
Author: vinkwok <[email protected]>
Date:   Thu Aug 11 17:27:31 2022 +0800

    Lock DiscordKit commit hash

    In progress of debugging some GitHub actions issue. Also includes minor pixel-perfection improvements.

commit 87bb3f2
Author: vinkwok <[email protected]>
Date:   Thu Aug 11 17:10:39 2022 +0800

    Fix mergingWithPartialMsg: return new instance

    Self cannot be mutated since mutable functions don’t exist in classes, so return a new instance instead. This should fix this actions build failure.

commit 0cdb7f6
Author: vinkwok <[email protected]>
Date:   Thu Aug 11 14:53:57 2022 +0800

    Squashed commit of the following:

    commit c54920f
    Author: cxt <[email protected]>
    Date:   Mon Aug 8 18:23:17 2022 +0000

        Translated using Weblate (German)

        Currently translated at 98.1% (106 of 108 strings)

        Translation: Swiftcord/Swiftcord
        Translate-URL: https://hosted.weblate.org/projects/swiftcord/swiftcord/de/

    commit 81cde3d
    Author: jnghyk <[email protected]>
    Date:   Mon Aug 8 11:37:50 2022 +0000

        Translated using Weblate (Korean)

        Currently translated at 100.0% (82 of 82 strings)

        Translation: Swiftcord/Swiftcord
        Translate-URL: https://hosted.weblate.org/projects/swiftcord/swiftcord/ko/

    commit dfc2c03
    Author: jnghyk <[email protected]>
    Date:   Mon Aug 8 11:15:47 2022 +0000

        Translated using Weblate (Korean)

        Currently translated at 98.7% (81 of 82 strings)

        Translation: Swiftcord/Swiftcord
        Translate-URL: https://hosted.weblate.org/projects/swiftcord/swiftcord/ko/

    commit 8782877
    Author: vinkwok <[email protected]>
    Date:   Thu Aug 4 22:24:03 2022 +0800

        Initial reply UI

    commit 0b2fc65
    Author: vinkwok <[email protected]>
    Date:   Wed Aug 10 18:38:36 2022 +0800

        Optimisation: play/pause animation when required

        Instead of switching between a CachedImageView and SwiftyGifView

commit 5d29118
Author: vinkwok <[email protected]>
Date:   Thu Aug 11 14:48:29 2022 +0800

    Render replies to “action messages” properly

    Note: Every commit from now onwards expects that Message is a class and referenced_message is of type Message.

commit b4e22a0
Author: vinkwok <[email protected]>
Date:   Thu Aug 11 13:58:50 2022 +0800

    Put message rendering views in their own folder

commit bbd5e1b
Author: vinkwok <[email protected]>
Date:   Thu Aug 11 12:40:39 2022 +0800

    Support more message types!

    Newly supported types:
    * recipientAdd (member added to a group DM)
    * recipientRemove (member removed from group DM)
    * userPremiumGuildSub (server boosted)
    * userPremiumGuildSubTier1, userPremiumGuildSubTier2, userPremiumGuildSubTier3 (server boosted and achieved tiers 1, 2 or 3 respectively)

commit 36a59cd
Author: vinkwok <[email protected]>
Date:   Thu Aug 11 11:15:07 2022 +0800

    UI bugfixes and optimisations

    * Use BetterImageView where possible to increase conciseness
    * Remove animation from server button when pressed (like Discord)
    * Make channel icon in message list header align with the official client

commit 0a44232
Author: vinkwok <[email protected]>
Date:   Thu Aug 11 10:39:11 2022 +0800

    Major Keychain helper cleanup

    Run blocking operations on the utility DispatchQueue to prevent hanging the main thread

commit a05f652
Author: vinkwok <[email protected]>
Date:   Wed Aug 10 19:03:32 2022 +0800

    Fix README typo, summarise some points

commit 161bc02
Author: vinkwok <[email protected]>
Date:   Wed Aug 10 18:38:36 2022 +0800

    Optimisation: play/pause animation when required

    Instead of switching between a CachedImageView and SwiftyGifView

commit c34cb21
Author: vinkwok <[email protected]>
Date:   Wed Aug 10 13:23:18 2022 +0800

    Allow SwiftyGifView to be sized with .frame

    Supplying width and height arguments are no longer required
# Conflicts:
#	Swiftcord.xcodeproj/project.pbxproj
#	Swiftcord.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
#	Swiftcord/Utils/Extensions/MessagesView+.swift
#	Swiftcord/Views/Message/MessageRenderViews/MessageView.swift
#	Swiftcord/Views/Message/MessagesView.swift
@cryptoAlgorithm cryptoAlgorithm merged commit fa10cdc into main Aug 11, 2022
@cryptoAlgorithm cryptoAlgorithm deleted the reply branch August 11, 2022 12:49
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