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

Svelte 1.6 #69

Merged
merged 132 commits into from
Jun 9, 2023
Merged

Svelte 1.6 #69

merged 132 commits into from
Jun 9, 2023

Conversation

qbjl
Copy link
Contributor

@qbjl qbjl commented Jan 10, 2023

Draft pull request for Meower 1.6

@qbjl qbjl added the release A release (for pull requests that merge develop into master) label Jan 10, 2023
@qbjl qbjl marked this pull request as draft January 10, 2023 17:51
@qbjl
Copy link
Contributor Author

qbjl commented Jan 10, 2023

PostList PR needs to still be finished, + fix "add more pfps" PR to add to develop

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 10, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1f8784b
Status: ✅  Deploy successful!
Preview URL: https://1c27c795.meower-svelte.pages.dev
Branch Preview URL: https://develop.meower-svelte.pages.dev

View logs

@showierdata9978
Copy link
Member

Also blocked by #71

@JoshAtticus

This comment was marked as off-topic.

@CST1229 CST1229 marked this pull request as ready for review June 7, 2023 11:37
@CST1229
Copy link
Member

CST1229 commented Jun 7, 2023

still blocked by bgm-related server changes (to disable bgm for existing recent accounts, since svelte didn't have bgm before and it's enabled by default on the server), undrafting for reviews

@CST1229
Copy link
Member

CST1229 commented Jun 7, 2023

showier do we really need to have a bunch of new pfps when cl4 port is around the corner

5 months later:

@CST1229
Copy link
Member

CST1229 commented Jun 7, 2023

nice
image

Copy link
Member

@showierdata9978 showierdata9978 left a comment

Choose a reason for hiding this comment

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

other then @html yes

Copy link
Contributor

@MikeDev101 MikeDev101 left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Member

@Bloctans Bloctans left a comment

Choose a reason for hiding this comment

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

ok mikedev

Copy link
Member

@JoshAtticus JoshAtticus left a comment

Choose a reason for hiding this comment

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

sus

@Bloctans
Copy link
Member

Bloctans commented Jun 9, 2023

merge when???

@tnix100
Copy link
Member

tnix100 commented Jun 9, 2023

Friday (EDT), waiting for Mike to update BGM settings on the server so people don't get confused when BGM suddenly starts playing because they've never experienced BGM before.

…art of the post content (#152)

Tested, is working.
@tnix100 tnix100 removed the Blocked Blocked by an issue or pull request label Jun 9, 2023
@theycallhermax
Copy link
Contributor

other then @html yes

calm down showier it uses cst's escape hack geez

@MikeDev101
Copy link
Contributor

Server patches applied and BGM is now disabled by default, still need to apply BGM updates to recent-ish accounts.

@MikeDev101
Copy link
Contributor

Didn't have tnix available to know how to update newer-ish accounts, so I just turned off BGM for all accounts in the database. This PR should be ready to merge.

@qbjl
Copy link
Contributor Author

qbjl commented Jun 9, 2023

we should def merge, is everything like changelog ok??? we should make sure that everything is correct before we merge

we should also announce this lol :3

@MikeDev101
Copy link
Contributor

we should def merge, is everything like changelog ok??? we should make sure that everything is correct before we merge

we should also announce this lol :3

Changelog appears to have sufficient details. Going to make the announcement now.

@qbjl
Copy link
Contributor Author

qbjl commented Jun 9, 2023

ok i think everything looks good. will merge if everyone's fine

@qbjl
Copy link
Contributor Author

qbjl commented Jun 9, 2023

merging now!!!!!! yay

@qbjl qbjl merged commit 87ef42a into master Jun 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release A release (for pull requests that merge develop into master)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[all | >1.5] Longer sessions lead to performance/memory issues