Skip to content

Dylan/feat/menu server logs #36

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 279 commits into
base: master
Choose a base branch
from
Open

Dylan/feat/menu server logs #36

wants to merge 279 commits into from

Conversation

dylanh724
Copy link
Contributor

@dylanh724 dylanh724 commented Apr 30, 2024 •
edited
Loading

About

Get server logs, with customizations

image

Feats

  • Get server names via CLI
    • => Load into dropdown, set default from CLI default
  • Get identity names via CLI
    • => Load into dropdown, set default from CLI default
  • Get module name from Publisher Window, if any
  • Persistence: None, since logs are probably too big to save.
    • However, for future options, we should persist the pref

Prerequisites

  1. Convert Unity SDK into a CSharp SDK with Unity support com.clockworklabs.spacetimedbsdk#34

Please merge above 1st since this feat is branched from this open PR

TODO

  • Option wrap text = false
  • Option Show timestamps = true
  • UX: On click, disable btn -> show an action verb txt => then revert on done
  • Research the max char limits and replace with some buffer to let the user know it's truncated.
    • There may be none (I do know the console logs have a limited # of lines you can add)
  • UX: Unity's scrollbars natively don't support resizable windows (even EverQuest had that in ~2000), so instead, find a hacky way to add them

dylanh724 added 30 commits March 5, 2024 11:23
@dylanh724
fix!: Replace Plastic Newtonsoft with general one
d086c2f 
- You wouldn't have noticed this error unless you clone freshly into a new project
@dylanh724
Merge branch 'master' into dylan/publisher-editor-tool
db54a83
@dylanh724
Merge branch 'dylan/publisher-editor-tool' into dylan/feat/reducers-e…
28c358d 
…ditor-tool
@dylanh724
Merge branch 'dylan/feat/reducers-editor-tool'
c7e1cb1
@dylanh724
fix!: When 1st creating, fix false positive fail
bc92b61
@dylanh724
Merge branch 'dylan/publisher-editor-tool' into dylan/feat/reducers-e…
abb3264 
…ditor-tool
@dylanh724
fix!: Handles deep nested reducers, custom types:
af0ffe1 
- fix: Originally causing breaking err, this now handles custom types and deep nested, currently swapping objects for simply `{}` or `SomeTypeName[]` for arrays
  - ux: I was trying to get the custom obj type name, but I only get the `Ref` number that doesn't seem to actually ref anything in `spacetime describe`
- ux: Removed Focus() for selecting an element for easy keyboard up+down through the list
- ux: Wrapped <nobr> around each syntax hint and allow wrapping -- for long args[]
- ux: Added more known types and parsed to C# type
- feat: Added CustomRefNum ("Ref") to EntityStructure
-
@dylanh724
Merge branch 'dylan/feat/reducers-editor-tool'
6a77f8b
@dylanh724
doc(README): Fork info
d393816
@dylanh724
chore: Null err checks for pub [un]subscribe events
e889bf3
@dylanh724
chore: Null err checks for pub [un]subscribe events
ce565b6
@dylanh724
chore: Null err checks for pub [un]subscribe events
98ddad2
@dylanh724
chore: Null err checks for reducer[un]sub events
7eb07d5
@dylanh724
Merge branch 'dylan/feat/reducers-editor-tool'
5e079ad
@dylanh724
feat: Wipe token playerPref on new connect address
ef6f6fc
@dylanh724
Merge branch 'master' of https://github.com/dylanh724/com.clockworkla…
7b28823 
…bs.spacetimedbsdk
@dylanh724
Merge branch 'master' into dev
dcbb361
@dylanh724
feat: "as identity"
c2b4f5d
@dylanh724
Merge branch 'dylan/feat/reducers-editor-tool'
e59a6ed
@dylanh724
Merge branch 'master' into dev
1a11fca
@dylanh724
Revert "Merge branch 'dylan/feat/reducers-editor-tool'"
76c882b 
This reverts commit e59a6ed, reversing
changes made to 7b28823.
@dylanh724
Merge branch 'master' into dylan/publisher-editor-tool
dbc0efd
@dylanh724
feat: "Clear all data?" toggle
834e1de 
- Default off, adds -c when true
@dylanh724
feat: "Clear all data?" toggle
83d75ea 
- Default off, adds -c when true
@dylanh724
Merge branch 'dylan/feat/reducers-editor-tool' into dylan/publisher-e…
1faea43 
…ditor-tool
@dylanh724
fix!: Remove inadvertent merge from reducers
7712def
@dylanh724
Merge branch 'dylan/publisher-editor-tool' into dev
fdb409f
@dylanh724
fix: Show minimized cached result on init, if exists
0f2d7cd
@dylanh724
fix: Overlapping formatting due to backslash paths
caf8890
@dylanh724
fix: CliOutput should just log info, not err
aed49e6
dylanh724 added 17 commits April 30, 2024 13:04
@dylanh724
doc: TODO comments for _argSnippets
23f0b95
@dylanh724
feat: Upgrade id txt into dropdown
70494c9 
- Pulls from Publisher
- Defaults to default identity, if none previously selected
- Will persist previously selected (and will soft check if still valid)
@dylanh724
feat: Persist selected reducer
53c4840
@dylanh724
Merge branch 'dylan/feat/reducers-editor-tool' into dylan/feat/menu-i…
4e1f5f5 
…nit-new-module
@dylanh724
feat: Init uxml, uss, readme
0d85009
@dylanh724
chore: Minor SpacetimeMeta mv using to top
7c357f3
@dylanh724
chore: Minor SpacetimeMeta mv using to top
c5d9cd4
@dylanh724
Merge branch 'dylan/publisher-editor-tool' into dylan/feat/reducers-e…
6279dd5 
…ditor-tool
@dylanh724
Merge branch 'dylan/feat/reducers-editor-tool' into dylan/feat/menu-s…
cdd91f9 
…erver-logs
@dylanh724
feat: ServerLogViewerMeta.cs
3418d46
@dylanh724
feat: ServerLogViewerWindow.cs
b6348d4
@dylanh724
feat: ServerLogViewerWindowCallbacks
fd76d6f
@dylanh724
feat: Server dropdown opt, ServerLogViewerAction
934f828
@dylanh724
doc(pub): Minor var name typo fix
7f16e3b
@dylanh724
chore: Mv Publish log styler to common
c907d3f
@dylanh724
feat: Add more opts to cli get logs
b1e68f8
@dylanh724
feat: Finish get logs implementation (+horiz scroll)
2a39ae6
@dylanh724 dylanh724 added the enhancement New feature or request label Apr 30, 2024
@jdetter jdetter removed their request for review November 8, 2024 18:10
@bfops bfops assigned bfops and unassigned jdetter Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bfops bfops Awaiting requested review from bfops

@cloutiertyler cloutiertyler Awaiting requested review from cloutiertyler

At least 1 approving review is required to merge this pull request.

Assignees

@bfops bfops

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dylanh724 @bfops @jdetter