Skip to content
This repository has been archived by the owner on Aug 25, 2023. It is now read-only.

Typos and more #44

Open
MattMacs opened this issue Mar 8, 2020 · 14 comments
Open

Typos and more #44

MattMacs opened this issue Mar 8, 2020 · 14 comments
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@MattMacs
Copy link
Contributor

MattMacs commented Mar 8, 2020

With the current Dockerfile the build fails at
https://github.com/IUNetSci/BotSlayer-CE/blob/master/Dockerfile#L22
--branch major_update

Removing this part works, but the config page is not available...

@phui
Copy link
Contributor

phui commented Mar 8, 2020

Thanks for letting us know. It was a leftover from deploy test.

The config page is removed on this update. This is to avoid issues with exposing the config setup to public.

This open-source version is for developers. We believe users are able to interact with the database directly. Users can set queries and keys by updating the config table in the PostgreSQL database.

Alternatively, please consider using the Pro version of BotSlayer, where the users still have access to the config page.

@phui
Copy link
Contributor

phui commented Mar 8, 2020

Fixed with eb673ed.

@phui phui closed this as completed Mar 8, 2020
@phui phui self-assigned this Mar 8, 2020
@MattMacs
Copy link
Contributor Author

MattMacs commented Mar 8, 2020

Then you also should update the frontend
"If the table still doesn't appear below, check the Config page to see that the Twitter keys match."

@filmenczer filmenczer reopened this Mar 8, 2020
marcmccarty added a commit that referenced this issue Mar 8, 2020
@marcmccarty
Copy link
Contributor

marcmccarty commented Mar 8, 2020

(Commit e008e61)
If this suitably solves the discrepancy, please close the issue.

@filmenczer
Copy link
Member

Don't we also need to update the Dockerfile per above?

@marcmccarty
Copy link
Contributor

Commit eb673ed per @phui

@filmenczer
Copy link
Member

Ah thanks. Oh and also the README still refers to the Config page...

@marcmccarty
Copy link
Contributor

Good catch. Should be fixed via:
Commit 13f8af0

@filmenczer
Copy link
Member

Hmm, there is no SAVE button, the config table is to be modified via psql...

@marcmccarty
Copy link
Contributor

Commit 7d617c6 for SAVE reference

@MattMacs
Copy link
Contributor Author

MattMacs commented Mar 8, 2020

in README.md:
delete: There are different rows for different types of qieries.
set: There are different rows for different types of queries.

@MattMacs
Copy link
Contributor Author

MattMacs commented Mar 8, 2020

The picture in README.md also still reflects the CONFIG page

@phui
Copy link
Contributor

phui commented Mar 9, 2020

@MattMacs Thanks for following up!

The typo is fixed by 6bf68b2

I will upload a new image after lunch.

@phui phui added the documentation Improvements or additions to documentation label Mar 9, 2020
@MattMacs MattMacs changed the title Docker build fails again, Config page not available Typos Mar 18, 2020
@MattMacs MattMacs changed the title Typos Typos and more Mar 18, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

4 participants