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

4.3.5 - prep overview #281

Closed
5 of 6 tasks
FrutyX opened this issue Oct 25, 2021 · 51 comments
Closed
5 of 6 tasks

4.3.5 - prep overview #281

FrutyX opened this issue Oct 25, 2021 · 51 comments

Comments

@FrutyX
Copy link
Collaborator

FrutyX commented Oct 25, 2021

(Last checked: 1.1.2022)

High priority

None.

Medium priority

None.

Low priority

None.

FINISHED

@GramThanos
Copy link
Member

Hey! I just wanted to let you know that I agree and that this is helpful.
I haven't replied yet because I am in the middle of moving house so I don't have too much free time these weeks.

@GramThanos GramThanos self-assigned this Oct 26, 2021
@GramThanos GramThanos pinned this issue Oct 27, 2021
@FrutyX
Copy link
Collaborator Author

FrutyX commented Oct 28, 2021

I see, that's understandable, don't want to push you, just a reminder GCA still exists :D :).

  • List updated.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Oct 28, 2021

@GreatApo looks like your fix bears fruit (#257) , which is nice. Don't forget to add it on the progress list since you closed it already.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Nov 2, 2021

Sorry Gram, I know you are moving, but 2:0 for Apo! 😛

@GramThanos
Copy link
Member

GramThanos commented Nov 2, 2021

This week I am in Spain for work. :P and there is still some packaging 📦 left to be done back in Greece.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Nov 3, 2021

Wish I could help with that damn import/export notes problem (language), but that script is waaaaaaay too complicated for me.

No rush tho, enjoy Spain, try some local food :).

@GreatApo GreatApo closed this as completed Nov 3, 2021
@GreatApo GreatApo reopened this Nov 3, 2021
@FrutyX
Copy link
Collaborator Author

FrutyX commented Nov 3, 2021

dog-surprised

@FrutyX
Copy link
Collaborator Author

FrutyX commented Nov 10, 2021

Changed #248 from TESTING to FIXED ,-).

@FrutyX
Copy link
Collaborator Author

FrutyX commented Nov 10, 2021

I don't usually don't dig into personal things, but curiosity is my weakness (will probably kill me one day)...

After seeing your LinkedIn profiles, shit, it does make sense that you don't have time for some dead browser game, lmao.

And, I have to ask, u guys are brothers? By any chance? I could not resist.

@GreatApo
Copy link
Member

Yeah, we are twins :P

Thanos takes care of the release the last couple of years. Normally, we would have push for a release because as you said, there are too many features/fixes. However, as he mentioned he is into a transition period now and he is mostly offline :)

@FrutyX
Copy link
Collaborator Author

FrutyX commented Nov 11, 2021

Twins, yeeeeey :D.

And ye, that's fine, I've just picked the wrong time to be active here, when you are using something daily, you clearly want to improve it, despite we all know in what state Gladiatus is, and it's future, it does not mean we should just give up and throw it into a trashcan.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Nov 13, 2021

Changed (again) #248 to PROBLEMATIC (more details in it's issue).

rough-day-keyboard

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 8, 2021

Issue reworked, list reworked, list updated, removed all my other talk, so it is more clear.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 13, 2021

@GramThanos Hey man, I very appreciate the invitation and I did accept it, but I have to mention, don't have that big expectations, I'll gladly help with anything I can, but my knowledge is basic, and I have more experience with running a website, than doing mostly javascript, to shorten it, if I can do it, I'll do it, if I can't do it, I'll just sit like a duck.

Thank you.

@GramThanos
Copy link
Member

@FrutyX no problem man, I invited you because you are already helping.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 21, 2021

@GramThanos, @GreatApo

Don't worry about the stuff I do directly to master now, just something I am preparing, for the documentation stuff.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 21, 2021

So, I gotta ask guys, can we add the Troubleshooting guide to the main GitHub page and to this:

image

I mean, I can add it, but I better ask if you agree with it, don't want to overdo my permissions.

Also I have zero feedback on the guide itself, so if you dislike something... just name it.

@GramThanos
Copy link
Member

@FrutyX awesome. Yes, good idea, if you have time please do add it.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 22, 2021

Talkin' about priorities Gram... there is still #279 and #235 without any progress, you got some catching up for Christmas 😛

@GramThanos
Copy link
Member

Aaaaaaa, low priority these are ...
Learn you will...
yoda-star-wars

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 22, 2021

Damn, but they are not on the list without a reason.... I gotta find a different developer 😄.

Hmmm, Apooooooooooo? 🔊

@GreatApo
Copy link
Member

GreatApo commented Dec 22, 2021

Damn, but they are not on the list without a reason.... I gotta find a different developer 😄.

Hmmm, Apooooooooooo? 🔊

Thanos spend 5 years studying computer science but failed to identify a string/number sorting bug...

image

@GramThanos
Copy link
Member

The auction sort needs to be merged with the advance packages filtering code as they solve similar issues. It is not so simple.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 22, 2021

Oh yes, this sounds like a good plan, I'm gonna make this a little sport and brother rivalry and GCA will prosper 🤣.

@GramThanos
Copy link
Member

I am not the one using parseInt without a base...

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 22, 2021

List updated, better take a look Gram! :D

Well, a few days will be needed to test everything in one package, #248 will need more time for testing too, #279 should probably be addressed too before a release.

And there are some little things like polishing stuff, but we are getting very close to a stable release.

Edit: Good job, both of you, and thanks.

@GreatApo
Copy link
Member

We probably need to make a release before the new year and the next one next year

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 22, 2021

@GramThanos For the custom market labels, the value label is not translated, I probably can add it to the code, but I was thinking, maybe you can get it directly from the game (UI)? Every item has a value, and I remember you did that with Threat in a different issue.

value

Edit: From looking at it, it might be more difficult, than I think, huh?

@GramThanos
Copy link
Member

This is already there, but needs improvement. I fixed it but in the wrong branch, I will have to move my changes on the correct one.

@GramThanos
Copy link
Member

Fix pushed 7b3fdc6. Check if it works now, this is server dependent.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 23, 2021

Still the same. Maybe it would be easier to just add the locale in the end.

value

Edit: Ooooh wait, It changed when I moved an item there and it seems it's staying there, so like other GCA things, it had to get loaded somehow.

Now after checking the code, I see you made it that way.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 24, 2021

"It's Christmas..."

And Apo took it personally:

Apo

@GramThanos take notes 🤣

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 24, 2021

@GreatApo

Výstřižek

Didn't Skars had a calculator on his website?

@GreatApo
Copy link
Member

@GreatApo

Výstřižek

Didn't Skars had a calculator on his website?

Has he? Don't remember 😂
I am trying to calculate the gold your guild has stolen (because you want me to) by calculating all the gold spend (assuming 30% discount).

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 24, 2021

https://gladiatus.gamerz-bg.com/calculator
Scroll down, check the source code, it's right there, it was pretty reliable if I remember it correctly.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 24, 2021

(because you want me to)

Don't act like it's for me, I am kinda surprised you focused on this feature request, originally it came from my guild mate, not from me, but I do agree it would be useful, also you have a good idea for it.

A lot has changed since I made that request, I am not in that guild anymore, what I am trying to say is, that even if you make it possible, I will not be able to test it, but I am not saying you should not make it.

@GreatApo
Copy link
Member

GreatApo commented Dec 24, 2021

(because you want me to)

Don't act like it's for me, I am kinda surprised you focused on this feature request, originally it came from my guild mate, not from me, but I do agree it would be useful, also you have a good idea for it.

A lot has changed since I made that request, I am not in that guild anymore, what I am trying to say is, that even if you make it possible, I will not be able to test it, but I am not saying you should not make it.

We can make it work for everyone except that guild... hahaha
I just had a look, I haven't script anything.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 24, 2021

I smell the sweet taste of progress in the air! List updated.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 25, 2021

The auction sort needs to be merged with the advance packages filtering code as they solve similar issues. It is not so simple.

I've never been actually using the packages filter, but I agree on this.

I see there are some Issues for packages filtering, and I was thinking, would the thing @GreatApo made in #225 also work with the packages filter, with some modifications of course?

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 25, 2021

@GreatApo , @GramThanos

Boys, I have been planning to create a new issue, a talk one. Sometimes I do comment and ask you things in different Issues, and this is not ideal, for example, I've been using this Issue to comment about something (and I am doing it right now too), so that's why I would like to create such Issue that will be pinned so it is visible, the idea (template for it) is here:

TALK - Only GCA related

Sometimes, you just have a simple question, or you want to suggest something, but it isn't completely up to create a whole new issue, this is the right place to talk about it.

Rules:

  1. As the name suggest, only GCA related talk here, Gladiatus topic is still tolerable, nothing else other than that.
  2. No vulgarity or spam, keep this Issue clean and when you forget to mention something, just edit your previous post.
  3. If you found a bug or want to make a suggestion for a feature or enhancement, create a New issue.
  4. If you are having problems with GCA, make sure to read our Troubleshooting guide first.
  5. You are free to use emojis and upload images, all in a reasonable amount.

In case of need, more rules might get added. Comments that will not respect these rules will be deleted.

You both good with this?

@GramThanos
Copy link
Member

I don't have a problem. Alternatively, we can create a Discord server.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 25, 2021

I've been thinking about a Discord server, buuuuuuuuut, I am not sure it will get any popular (we get people on it), like GitHub is our main platform now, so I am not sure if we also get any people on a discord server.

But that's just my opinion, maybe I am wrong.

@GramThanos
Copy link
Member

I expect 4 people to join.

@GreatApo
Copy link
Member

Guys, I track the GCA post in the Gladiatus Forum from time to time. Whoever wants to reach us, they can reach us. There is no reason to create more stuff, the player base is extremely small.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 25, 2021

@GreatApo and about the talk issue? That's the thing, I don't want to comment Issues to tell you something, when it's not related to the Issue, and as you see, for now, I use this Issue for it, but when it's done, I will close it.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 31, 2021

2 more days for #248 testing, but there are some bugs, so I don't know if we should fix them before a release. Current feature requests you are working on needs to be finished and polished before a release, so nothing is broken.

It seems like the release is getting further and further postponed. On one hand, you should release, because it's about time, on the other, you shouldn't, because I was wrong, when I thought we'll make it bug/error free, nothing is ever bug free.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 31, 2021

A note I would like to mention.

I don't know how much work is to release a version, I mean, how long does it take for browsers (checking process) to release it on their stores.

We could make smaller updates (hell, even hotfixes) but I guess it's too much work (changing everything on GitHub, changing it in the addon, changing it on your website) ...

@GreatApo
Copy link
Member

A note I would like to mention.

I don't know how much work is to release a version, I mean, how long does it take for browsers (checking process) to release it on their stores.

We could make smaller updates (hell, even hotfixes) but I guess it's too much work (changing everything on GitHub, changing it in the addon, changing it on your website) ...

Thanos has set everything up so that we can easily create the addon for multiple browsers. One problem is that Chrome has updated their manifest version from 2 to 3 which we need to update. The review time depends on the store. I think chrome is quite fast/automatic and it takes a day (we will see).
Nowadays we don't have huge bugs, everything that we find just needs small fixes. We usually try to release with all the identified bugs fixed.

@FrutyX
Copy link
Collaborator Author

FrutyX commented Dec 31, 2021

Did he just did it? God be with us.

Btw - Happy new year you two :)

@GramThanos
Copy link
Member

Happy new year!

@FrutyX FrutyX closed this as completed Jan 1, 2022
@FrutyX FrutyX unpinned this issue Jan 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants