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

Fix incorrect PostgreSQL data path in example doc #7942

Merged
merged 1 commit into from Sep 29, 2020

Conversation

Igi4
Copy link
Contributor

@Igi4 Igi4 commented Sep 17, 2020

Description of what you did:

Fixed the PostgreSQL data path to a correct one (postgres Docker image stores the database files in /var/lib/postgresql/data. The example documentation originally stated /data/postgres path, which is incorrect).

@codecov
Copy link

codecov bot commented Sep 17, 2020

Codecov Report

Merging #7942 into documentation will increase coverage by 13.57%.
The diff coverage is n/a.

Impacted file tree graph

@@                Coverage Diff                 @@
##           documentation    #7942       +/-   ##
==================================================
+ Coverage          19.14%   32.71%   +13.57%     
==================================================
  Files                855     1194      +339     
  Lines              11933    12965     +1032     
  Branches            1898     1279      -619     
==================================================
+ Hits                2284     4241     +1957     
+ Misses              8100     7884      -216     
+ Partials            1549      840      -709     
Flag Coverage Δ
#front 24.81% <ø> (+10.32%) ⬆️
#unit 53.85% <ø> (+13.92%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
packages/strapi-plugin-email/admin/src/pluginId.js 0.00% <0.00%> (-100.00%) ⬇️
...api-plugin-users-permissions/admin/src/pluginId.js 0.00% <0.00%> (-100.00%) ⬇️
...n/src/containers/NotificationProvider/constants.js 0.00% <0.00%> (-100.00%) ⬇️
...n/admin/src/containers/LanguageProvider/reducer.js 0.00% <0.00%> (-61.54%) ⬇️
packages/strapi/lib/services/metrics/index.js 65.00% <0.00%> (-35.00%) ⬇️
...admin/admin/src/components/ContainerFluid/index.js 75.00% <0.00%> (-25.00%) ⬇️
...trapi-admin/admin/src/containers/LeftMenu/index.js 42.30% <0.00%> (-24.36%) ⬇️
...in/admin/src/containers/Webhooks/EditView/index.js 20.33% <0.00%> (-18.07%) ⬇️
...-admin/admin/src/containers/HomePage/components.js 40.00% <0.00%> (-17.15%) ⬇️
...in-upload/admin/src/containers/HomePage/reducer.js 69.56% <0.00%> (-14.81%) ⬇️
... and 1266 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4eb7f65...781fba0. Read the comment docs.

@Igi4 Igi4 force-pushed the patch-1 branch 3 times, most recently from 6f16f92 to f281b07 Compare September 17, 2020 19:26
`postgresql` Docker image stores the database files in `/var/lib/postgresql/data`.

Signed-off-by: Igor Liška <liska.igor4@gmail.com>
@alexandrebodin alexandrebodin added this to the 3.1.7 milestone Sep 29, 2020
@alexandrebodin alexandrebodin added source: docs Documentation changes issue: enhancement Issue suggesting an enhancement to an existing feature labels Sep 29, 2020
@alexandrebodin
Copy link
Member

Thansk for the fix ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue: enhancement Issue suggesting an enhancement to an existing feature source: docs Documentation changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants