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

DataTable'd pogoauth settings #1375

Merged
merged 1 commit into from
Jan 5, 2024
Merged

Conversation

JabLuszko
Copy link
Contributor

@JabLuszko JabLuszko commented Jan 5, 2024

50 results per page, sorted by device column desc (so accounts tied to devices first in results).
Plus as bonus we get searchbar :D

50 results per page, sorted by device column desc (so accounts tied to devices first in results).
@Grennith Grennith merged commit 0777dd2 into Map-A-Droid:async Jan 5, 2024
3 of 7 checks passed
@mpw1337
Copy link

mpw1337 commented Jan 9, 2024

Isn't this missing a table id for the pgoauth table itself? It only adds the JS but the id is never given to the table? Or am I missing something? @JabLuszko

<table class="table table-striped table-hover table-sm">
      <thead>
        <tr>
          <th style="width: 15%" class="align-middle">Type <i class="fas fa-info-circle" data-toggle="tooltip" title="Login Type"></th>
          <th style="width: 30%" class="align-middle">Username <i class="fas fa-info-circle" data-toggle="tooltip" title="Username for the login"></th>
          <th style="width: 30%" class="align-middle">Device <i class="fas fa-info-circle" data-toggle="tooltip" title="Device assigned to this login along with instance ID of the device"></th>
          <th style="width: 10%" class="text-center align-middle">Level <i class="fas fa-info-circle" data-toggle="tooltip" title="Account in-game level, 8 for raids, 30 for mons/quests"></th>
          <th style="width: 5%" class="text-center align-middle"></th>
          <th style="width: 10%" class="text-right align-middle"><a href='{{ url('settings_pogoauth', query_={'id': 'new'}) }}'><button type="button" class="btn btn-sm btn-success"><i class="fas fa-plus"></i></button></a></th>
        </tr>
      </thead>

@JabLuszko
Copy link
Contributor Author

Totally true @mpw1337 , fucked up while copy-pasting, thanks!
Welp, another PR :D

Grennith pushed a commit that referenced this pull request Jan 14, 2024
50 results per page, sorted by device column desc (so accounts tied to devices first in results).
Grennith pushed a commit that referenced this pull request Jan 15, 2024
* Update settings_pogoauth.html

* Update settings_pogoauth.html
Grennith pushed a commit that referenced this pull request Mar 31, 2024
* Update settings_pogoauth.html

* Update settings_pogoauth.html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants