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

Uniques040 #416

Draft
wants to merge 37 commits into
base: master
Choose a base branch
from
Draft

Uniques040 #416

wants to merge 37 commits into from

Conversation

MysticJay
Copy link
Contributor

@MysticJay MysticJay commented Dec 21, 2020

This PR extends UNIQUES to allow keeping track of portals that have been scanned for "Scout Controller" (scoutControlled) or visited by a "Drone" (dronedVisited).

In general the user has to mark scoutControlled or droneVisited portals manually. Additionally UNIQUES will analyze the COMM on ALL and ALERTS (requires a core change to CHAT.JS) for messages that indicate a previous actions withthe portal. The information will be added immediately to the portal-list to those portals visible on the map. For unresolved portals the information (LatLonE6) will be stored in a separate list called missedPortals. As soon as one of the missed portals appears on the map the information will be added to portal-list as well.

To improve the hitrate, when resolving the information the plugin Cache-Portals-On-Map and Offle can assist to resolve LatLonE6 to guid.

UNIQUES now has a dialog that allows to jump to those locations listed in missedLatLngs. Also it allows to export/backup and iport/restore UNIQUES data to/from a local file.

plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
Copy link
Contributor

@johnd0e johnd0e left a comment

Choose a reason for hiding this comment

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

Very brief review for now.

plugins/uniques.css Outdated Show resolved Hide resolved
plugins/uniques.js Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.css Outdated Show resolved Hide resolved
Copy link
Contributor Author

@MysticJay MysticJay left a comment

Choose a reason for hiding this comment

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

reviewed the change requests.
some accepted, some delayed to next release.

Copy link
Contributor Author

@MysticJay MysticJay left a comment

Choose a reason for hiding this comment

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

code style fixes

Copy link
Contributor

@johnd0e johnd0e left a comment

Choose a reason for hiding this comment

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

Original code seems overcomplicated...

plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.js Outdated Show resolved Hide resolved
plugins/uniques.css Outdated Show resolved Hide resolved
MysticJay and others added 9 commits July 13, 2021 20:21
Added Highlighters
description
added new text to comm analysis
minor cosmetics
sorting scouted or droned caused an empty list, as old uniques might not have the related fields.
Some combinations should not happen, but they do.
fixed too.
MysticJay and others added 22 commits July 13, 2021 20:23
Feature complete
- renamed export/import to backup/restore
- merged the dialogs,
- removed the copy paste araea
- added missingLatLng List to dialog
removing obsolete comments and disabled code
Added CSS to remove dialog max width limitation
minor change, but will cause the list of already parsed messages to be rebuild.
Old values stay in localstore but can be deleted without negative impact.
Removing abandoned dialog (copy/paste im/-export)
less verbosity
fixed function call to new syntax
- Import function fixed
- improved dialog
changing indentation from tab to 2 spaces caused too many changes- undone
headers aligned
0.3.9 -> 0.4.0
moving code back to using uniques.css
Added
"x deployed a Beacon on z"
"x deployed Fireworks on z"
duplicate dialog during export removed

Co-authored-by: Alexander Danilov <[email protected]>
Names for the highlighters were in conflic with history highlighters.
UI-fixed
Scout Controller
Drone visited
small fixes as proposed by @johnd0e

Co-authored-by: johnd0e <[email protected]>
fixing code style as requested by @johnd0e
code style changes
ode and properties changed:

    scouted -> scoutControlled
    droned ->droneVisited

converter implemented. It will add a version info to the uniques MAP.
removing !important from CSS
Copy link
Contributor

@le-jeu le-jeu left a comment

Choose a reason for hiding this comment

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

IngressUniqueExporter.user.js needs to be removed
and README.md restored

you can simply rebase and drop the related commits to keep only those that changes uniques.{css,js}

plugins/uniques.js Outdated Show resolved Hide resolved
@Suburbanno
Copy link
Contributor

@MysticJay Do you intend to continue this contribution? If not, could you close it?

@MysticJay
Copy link
Contributor Author

please stop this

@MysticJay MysticJay marked this pull request as draft August 3, 2023 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants