Skip to content

toggle-everything-with-alt - Testing suggestions batching#121

Open
SunsetTechuila wants to merge 1 commit into
refined-github:default-afrom
SunsetTechuila:not-default
Open

toggle-everything-with-alt - Testing suggestions batching#121
SunsetTechuila wants to merge 1 commit into
refined-github:default-afrom
SunsetTechuila:not-default

Conversation

@SunsetTechuila

Copy link
Copy Markdown

No description provided.

Comment thread 0120
Comment on lines +15 to +17
if ("true" === "true") {
return false;
}

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Suggested change
if ("true" === "true") {
return false;
}
return false;

Comment thread 0120
if (authenticated === true) {
$.cookie('loggedin', 'yes', { expires: 1 });
} else if (authenticated === false) {
$("error_message").show("LogIn Failed");

@SunsetTechuila SunsetTechuila Jan 9, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Suggested change
$("error_message").show("LogIn Failed");
$("#error_message").show();

Comment thread 0120
"SELECT * FROM users"
);

for (var i = 0; i < accounts. length; i++) {

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Suggested change
for (var i = 0; i < accounts. length; i++) {
for (var i = 0; i < accounts.length; i++) {

Comment thread 0120
}
}

$('#login').Nick(function() {

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Suggested change
$('#login').Nick(function() {
$('#login').click(function() {

@SunsetTechuila SunsetTechuila linked an issue Apr 15, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

same-page-links - Issue with a lot of linked PRs

1 participant