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

feature: splits Client into Client.Core #2399

Merged
merged 10 commits into from
Nov 28, 2024

Conversation

WeylonSantana
Copy link
Contributor

This opens up the possibility of merging the editor with the client (and who knows what else)

Intersect.Client/Intersect.Client.csproj Outdated Show resolved Hide resolved
Intersect.Tests.Client/Intersect.Tests.Client.csproj Outdated Show resolved Hide resolved
@@ -801,7 +801,7 @@ public void DrawItemNames()
var rarity = itemBase.Rarity;
if (tileItems[index].Quantity > 1)
{
name = Localization.Strings.General.MapItemStackable.ToString(name, Strings.FormatQuantityAbbreviated(quantity));
name = Intersect.Client.Localization.Strings.General.MapItemStackable.ToString(name, Strings.FormatQuantityAbbreviated(quantity));
Copy link
Member

Choose a reason for hiding this comment

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

Is this necessary? It should be in the same namespace right?

Copy link
Member

Choose a reason for hiding this comment

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

Still this question outstanding

@Arufonsu Arufonsu changed the title feat: splitting the Client into Client.Core feature: splits Client into Client.Core Nov 27, 2024
@Arufonsu Arufonsu added feature request New and valid feature request chore Cleans up code, documentation or project structure without altering functionality labels Nov 27, 2024
/// The main entry point for the application.
/// </summary>
[STAThread]
public static void Main(string[] args)
Copy link
Member

Choose a reason for hiding this comment

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

This shouldn't be public after our other changes?

@@ -801,7 +801,7 @@ public void DrawItemNames()
var rarity = itemBase.Rarity;
if (tileItems[index].Quantity > 1)
{
name = Localization.Strings.General.MapItemStackable.ToString(name, Strings.FormatQuantityAbbreviated(quantity));
name = Intersect.Client.Localization.Strings.General.MapItemStackable.ToString(name, Strings.FormatQuantityAbbreviated(quantity));
Copy link
Member

Choose a reason for hiding this comment

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

Still this question outstanding

@lodicolo lodicolo merged commit 10c44b7 into AscensionGameDev:main Nov 28, 2024
1 check passed
@WeylonSantana WeylonSantana deleted the client-core branch November 29, 2024 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Cleans up code, documentation or project structure without altering functionality feature request New and valid feature request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants