Skip to content

Add UA column #1447

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

Merged
merged 1 commit into from
Jun 25, 2018
Merged

Add UA column #1447

merged 1 commit into from
Jun 25, 2018

Conversation

Martii
Copy link
Member

@Martii Martii commented Jun 25, 2018

  • Fix some view elements too

NOTE:

  • This may need some mitigation as I'm quite tired twiddling. Haven't tested adding to existing login yet. At some point I'll boot everyone off for current sessions but not right now.

Post #1203 #1446 ... related to #604

* Fix some view elements too

NOTE:
* This may need some mitigation as I'm quite tired twiddling. Haven't tested adding to existing login yet. At some point I'll boot everyone off for current sessions but not right now.

Post OpenUserJS#1203 1446 ... related to 604
@Martii Martii added enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. UI Pertains inclusively to the User Interface. DB Pertains inclusively to the Database operations. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. labels Jun 25, 2018
@Martii Martii merged commit 982b5eb into OpenUserJS:master Jun 25, 2018
@Martii Martii deleted the addUA branch June 25, 2018 15:28
Martii added a commit to Martii/OpenUserJS.org that referenced this pull request Jun 26, 2018
* `getTimeStamp` isn't available to my knowledge on any identifiers since they are Objects of the real thing.

NOTE:
* Needed for a change that is being worked on

Post OpenUserJS#1203 OpenUserJS#1446 OpenUserJS#1447 ... related to OpenUserJS#604
@Martii Martii mentioned this pull request Jun 26, 2018
Martii added a commit that referenced this pull request Jun 26, 2018
* `getTimeStamp` isn't available to my knowledge on any identifiers since they are Objects of the real thing.

NOTE:
* Needed for a change that is being worked on

Post #1203 #1446 #1447 ... related to #604

Auto-merge
Martii added a commit to Martii/OpenUserJS.org that referenced this pull request Jun 26, 2018
* Tables and forms don't mix Duhh
* Add Sign Out routine... logged
* Bug fix for extending

NOTE:
* This isn't as pretty as I would like in View but still working on it
* Will be booting everyone off shortly for the bug fix

Post OpenUserJS#1203 OpenUserJS#1446 OpenUserJS#1447 ... related to OpenUserJS#604
Martii added a commit that referenced this pull request Jun 26, 2018
* Tables and forms don't mix Duhh
* Add Sign Out routine... logged
* Bug fix for extending

NOTE:
* This isn't as pretty as I would like in View but still working on it
* Will be booting everyone off shortly for the bug fix

Post #1203 #1446 #1447 ... related to #604

Auto-merge
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. DB Pertains inclusively to the Database operations. enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. UI Pertains inclusively to the User Interface.
Development

Successfully merging this pull request may close these issues.

1 participant