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

[Testing] Absolute Roleplay #3941

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

[Testing] Absolute Roleplay #3941

wants to merge 12 commits into from

Conversation

serifas
Copy link

@serifas serifas commented Jul 12, 2024

No description provided.

@bleatbot bleatbot enabled auto-merge (squash) July 12, 2024 03:48
@bleatbot
Copy link
Collaborator

bleatbot commented Jul 12, 2024

Outdated attempt

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

The average merge time for plugin updates is currently more than a day.

😰Absolute-Roleplay [testing-live] - db9cc5c - Build system error: DalamudPackager output not found, make sure it is installed

Show log - Review

@bleatbot bleatbot added new plugin This is a new plugin. build failed This plugin failed to build. labels Jul 12, 2024
auto-merge was automatically disabled July 12, 2024 04:07

Head branch was pushed to by a user without write access

@bleatbot
Copy link
Collaborator

bleatbot commented Jul 12, 2024

Outdated attempt

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

😰Absolute-Roleplay [testing-live] - 783ae6d - Build system error: DalamudPackager output not found, make sure it is installed

Show log - Review

@bleatbot
Copy link
Collaborator

bleatbot commented Jul 12, 2024

Outdated attempt

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

😰Absolute-Roleplay [testing-live] - 03ddee3 - Build system error: DalamudPackager output not found, make sure it is installed

Show log - Review

@bleatbot
Copy link
Collaborator

bleatbot commented Jul 12, 2024

Outdated attempt

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

😰Absolute-Roleplay [testing-live] - 340584e - Build system error: DalamudPackager output not found, make sure it is installed

Show log - Review

@bleatbot
Copy link
Collaborator

bleatbot commented Jul 12, 2024

Outdated attempt

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

✔️AbsoluteRoleplay [testing-live] - 340584e - v0.0.0.1 - Repo (New plugin)

Show log - Review

@bleatbot bleatbot added size-large Diff for this PR is large. and removed build failed This plugin failed to build. labels Jul 12, 2024
@reiichi001 reiichi001 added pending-code-review This plugin still needs code review. pending-rules-compliance This plugin still needs to be checked for rules compliance by the majority of the PAC. pending-testing This plugin still needs to be tested. labels Jul 12, 2024
Figured out there was a small chance of auto login not working and actually locking the connection for the user, This fixes it.
@bleatbot
Copy link
Collaborator

bleatbot commented Jul 17, 2024

Outdated attempt

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

✔️AbsoluteRoleplay [testing-live] - ba55129 - v0.0.0.1 - Repo (New plugin)

Show log - Review

@reiichi001
Copy link
Contributor

Testing results:

  • I would show a warning if the passwords do not match during registration. Currently, the plugin does not react. (the same applies to the forget password window)

  • If a user closes the verification window, there's no way to get it back. The plugin will just say the email is already in use.

  • A user can submit a forgot password before they have verified their account. (Is this intentional?)

  • After submitting information for account restoration, the window does not go away. It also has cut-off text.
    image

  • The wifi button by connected does not appear to do anything. Can you elaborate on what this is for?

  • the DTRBar entry might not actually be by the map. I would suggest rewording this to "server info bar" or "top information bar" or similar.

  • While logged in, the connections button does not seem to do anything

@serifas
Copy link
Author

serifas commented Jul 26, 2024

I will update the source with the changes. Just give me until probably tomorrow to submit it. Thank you for updating me.

@serifas
Copy link
Author

serifas commented Jul 26, 2024

also the way I have the registration setup with verification is working as I want it to asside from the message not displaying if passwords don't match, and the sizing issue.
The restoration window is intended to allow the user to request multiple times so if they close it they can just submit another request.
The connection button is for if the server goes down the users can manually reconnect. It doesn't really do anything if the server connection hasn't died.
As far as the DTRBar goes, are you talking about the hint I gave in the command to open the window?
I am fixing the window sizing and password messages tho. but the rest I believe is working correctly.

Set messages for passwords not matching and scaled the restoration window correctly, as well as moved away from config values when sending data to server as it relied on the user checking remember me.
@bleatbot
Copy link
Collaborator

bleatbot commented Jul 26, 2024

Outdated attempt

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

✔️AbsoluteRoleplay [testing-live] - 02c0683 - v0.0.0.1 - Repo (New plugin)

Show log - Review

Fixed an issue where the registration key would some times not work after registering.
@bleatbot
Copy link
Collaborator

bleatbot commented Jul 29, 2024

Outdated attempt

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

✔️AbsoluteRoleplay [testing-live] - add8e04 - v0.0.0.1 - Repo (New plugin)

Show log - Review

@serifas
Copy link
Author

serifas commented Sep 1, 2024

is this still being reviewed?

@reiichi001
Copy link
Contributor

is this still being reviewed?

It's still making its way through review. The PAC member assigned to code review has not had a lot of availability due to IRL taking priority. (And other PAC members who can do the code review portion have also been pretty busy.) I'll mention this in our internal review sections.

@serifas
Copy link
Author

serifas commented Sep 5, 2024

thank you, sorry, I didn't mean to come off as pestering <3

@bleatbot
Copy link
Collaborator

bleatbot commented Sep 21, 2024

Outdated attempt

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

✔️AbsoluteRoleplay [testing-live] - add8e04 - v0.0.0.1 - Repo (New plugin)

Show log - Review

@bleatbot
Copy link
Collaborator

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

✔️AbsoluteRoleplay [testing-live] - d02a5e0 - v0.0.0.1 - Repo (New plugin)

Show log - Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new plugin This is a new plugin. pending-code-review This plugin still needs code review. pending-rules-compliance This plugin still needs to be checked for rules compliance by the majority of the PAC. pending-testing This plugin still needs to be tested. size-large Diff for this PR is large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants