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

Add support in the GUI for new output types #5

Open
peterjaap opened this issue Oct 14, 2022 · 13 comments
Open

Add support in the GUI for new output types #5

peterjaap opened this issue Oct 14, 2022 · 13 comments

Comments

@peterjaap
Copy link
Contributor

peterjaap commented Oct 14, 2022

The amazing @convenient added a few output types to the Ampersand tool (see commit);

    const TYPE_QUEUE_CONSUMER_ADDED = 'Queue consumer added';
    const TYPE_QUEUE_CONSUMER_REMOVED = 'Queue consumer removed';
    const TYPE_QUEUE_CONSUMER_CHANGED = 'Queue consumer changed';
    const TYPE_DB_SCHEMA_ADDED = 'DB schema added';
    const TYPE_DB_SCHEMA_CHANGED = 'DB schema changed';
    const TYPE_DB_SCHEMA_REMOVED = 'DB schema removed';

Right now, this info is nowhere to be seen in the GUI, so we need to add it.

I'm terrible at UI, so maybe someone has an idea of where to show it. My thought would be to show an Info button on this page somewhere:

image

Maybe with an indicator of how many INFO notices there are. When clicking on it, you'll go to a screen that shows all the warnings?

Hint;

return line.indexOf('Preference') > 0 || line.indexOf('Plugin') > 0 || line.indexOf('Override') > 0;

@convenient
Copy link

@peterjaap DB schema are still WIP and subject to change 🙃

@peterjaap
Copy link
Contributor Author

@peterjaap
Copy link
Contributor Author

@peterjaap
Copy link
Contributor Author

Ok I've merged the work I've done into 1.3.1.

Notable changes;

  • Added a button INFO notices (X) (if X > 0) where X is the number of info notices
  • Upgraded Electron from v9 to v21
  • Upgraded electron-store
  • This upgrade fixed the developer console not showing up under Ubuntu 22.04
  • Installed Vue Development tools in development mode
  • Split the results.json file into warnings.json and infoNotices.json

The INFO notices button doesn't do anything yet except show an alert box saying it's not implemented yet;

image

Next step is to create an actual screen with the notices displayed.

@amenk
Copy link

amenk commented Nov 29, 2022

I hope that is related to this issue:

I see db_schema.xml output, but when I open it, the right site of the view does not load the db_schema.xml - instead the previously selected file stays at the right side.

Console prints:

vue.runtime.esm.js:1897 Error: The language "undefined" has no grammar.
    at highlight (prism-core.js:663:11)
    at o.customFileContentHighlighter (Editor.vue:215:14)
    at o.content (prismeditor.esm.js:124:25)
    at ir.get (vue.runtime.esm.js:4495:25)
    at ir.evaluate (vue.runtime.esm.js:4597:21)
    at o.content (vue.runtime.esm.js:4851:17)
    at o.render (prismeditor.esm.js:555:25)
    at e._render (vue.runtime.esm.js:3569:22)
    at o.r (vue.runtime.esm.js:4081:21)
    at ir.get (vue.runtime.esm.js:4495:25)

@peterjaap
Copy link
Contributor Author

Where do you see the db_schema.xml output? It shouldn't show up in the GUI anywhere at this point. Could you share a screenshot?

@amenk
Copy link

amenk commented Nov 29, 2022

Along with the other lines in the dropdown at the top...
Can share a screenshot tomorrow

@peterjaap
Copy link
Contributor Author

image

Found it. Fix incoming.

peterjaap added a commit that referenced this issue Nov 29, 2022
…loses #8)

Fix warning about missing grammar for wrong file extension (as mentioned in #5)
Auto open project directory by passing absolute path via command line (closes #6)
@amenk
Copy link

amenk commented Dec 2, 2022

Still having this with 1.3.2

image

same JS error

@peterjaap
Copy link
Contributor Author

@amenk fixed in 1.3.4

@peterjaap
Copy link
Contributor Author

@amenk amended in 1.3.5. I previously filtered out the "DB changed" WARN notices, but I re-added them and show a notice on the right hand side why no file is shown for "DB changed added" WARN notices;

No file found in previous install - this is a new file. See the diff on the left hand side to see the actual changes.

@amenk
Copy link

amenk commented Dec 9, 2022

Works well :-)

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

No branches or pull requests

3 participants