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

Shiny-R PostgresSQL Example #252

Merged
merged 8 commits into from
Jul 22, 2024
Merged

Shiny-R PostgresSQL Example #252

merged 8 commits into from
Jul 22, 2024

Conversation

stanleyo03
Copy link
Contributor

@stanleyo03 stanleyo03 commented Jul 18, 2024

Added example Shiny-R app that is connected to Postgres database.

Closes #253


📚 Documentation preview 📚: https://ploomber-doc--252.org.readthedocs.build/en/252/

@neelasha23
Copy link
Contributor

Please add Closes #issuenumber in the description @stanleyo03

@stanleyo03
Copy link
Contributor Author

stanleyo03 commented Jul 18, 2024

@neelasha23 There wasn't an issue for this PR, I was just adding the app I made for the blog post. Should I create an issue? Thanks!

@stanleyo03
Copy link
Contributor Author

Please add Closes #issuenumber in the description @stanleyo03

done!

@stanleyo03 stanleyo03 closed this Jul 18, 2024
@stanleyo03
Copy link
Contributor Author

closed temporarily to remove database info

@neelasha23
Copy link
Contributor

closed temporarily to remove database info

you can convert it to draft if still making changes, instead of closing

@edublancas edublancas deleted the Rshiny_postgres_example branch July 18, 2024 17:12
@stanleyo03 stanleyo03 restored the Rshiny_postgres_example branch July 18, 2024 17:13
@stanleyo03 stanleyo03 reopened this Jul 18, 2024
@stanleyo03 stanleyo03 marked this pull request as draft July 18, 2024 17:15
@neelasha23
Copy link
Contributor

Please tag and mark it as ready for review once it's ready

@stanleyo03 stanleyo03 marked this pull request as ready for review July 18, 2024 20:12
@stanleyo03
Copy link
Contributor Author

@neelasha23 ready for review


![](screenshot.png)

To use the app, fill in the database parameters with the parameters from your PostgreSQL database.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add the steps here or add a link to the blog once it's merged

@neelasha23
Copy link
Contributor

Please deploy this app to cloud and share the app link @stanleyo03

@stanleyo03
Copy link
Contributor Author

stanleyo03 commented Jul 22, 2024

@neelasha23 I think I want to change the sample dataset I'm using because I downloaded the data locally and don't want to upload it onto this repo. Thoughts? I will change the blog post to draft until I fix it.

@neelasha23
Copy link
Contributor

@neelasha23 I think I want to change the sample dataset I'm using because I downloaded the data locally and don't want to upload it onto this repo. Thoughts? I will change the blog post to draft until I fix it.

Didn't get it. Why do u need to upload to this repo? You can create a zip of the data and code files and deploy to Ploomber Cloud as a Docker deployment. am i missing something? @stanleyo03

@stanleyo03
Copy link
Contributor Author

stanleyo03 commented Jul 22, 2024

I noticed that from the ploomber cloud cli, you can deploy the example apps. However, my app wouldn't work because the data is local and not imported directly from an online source. But now that I'm thinking about it, users would have to add their database parameters, so they would have to modify the R files anyways. Should I just leave it as it is?

@neelasha23
Copy link
Contributor

neelasha23 commented Jul 22, 2024

You can try the Web UI: https://www.platform.ploomber.io/register/
we need to verify the app works on the cloud and document the exact steps so users can follow. You can add ur db credentials in the R file and upload the zip @stanleyo03

@stanleyo03
Copy link
Contributor Author

@neelasha23 here is a link to the app: https://postgres-shiny-app.dev.ploomberapp.io/

neelasha23
neelasha23 previously approved these changes Jul 22, 2024
@stanleyo03
Copy link
Contributor Author

@neelasha23 sorry could you approve this again, I noticed there was a missing closed parenthesis in one of the files

@edublancas edublancas merged commit 11298c7 into main Jul 22, 2024
1 check passed
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

Successfully merging this pull request may close these issues.

add Shiny-R with Postgres App example
3 participants