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

Re-implement the "import from Discourse" part in Discourse #13

Open
tanius opened this issue Oct 22, 2019 · 1 comment
Open

Re-implement the "import from Discourse" part in Discourse #13

tanius opened this issue Oct 22, 2019 · 1 comment

Comments

@tanius
Copy link
Member

tanius commented Oct 22, 2019

Since this API is a compact software and does not too much stuff at all, it seems preferable for its application together with Discourse to re-implement it as an extension of the Discourse API. This will have major performance improvements when updating the pre-generated files ("seconds, not hours") as a direct connection to the Discourse database and all of its internal caching in Redis is available. This would also make it possible to always analyze the real-time Discourse data.

In turn, the "import from Discourse" feature will no longer be needed in this software and could be removed. This software would then be a standalone implementation of the Graphryder API that can be used when one wants to analyze data not coming from Discourse but from some sort or generic, other data source.

After this, there would be some name changes:

  • This software would be renamed to graphryder-api-standalone.
  • The Discourse plugin, providing another implementation of the API, would be called graphryder-api-discourse.
@tanius
Copy link
Member Author

tanius commented Oct 23, 2019

Actually there is no need to remove the Discourse importer code from this software. It may still be useful for anyone wanting to analyze Discourse forum content of a forum they don't have admin control over (such as their own forum hosted with a SaaS solution, which usually means that they cannot install plugins there).

@tanius tanius changed the title Move the "import from Discourse" part to Discourse Re-implement the "import from Discourse" part in Discourse Oct 23, 2019
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

1 participant