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 homegrown / generic migration guide #2999

Merged
merged 19 commits into from
Sep 12, 2024

Conversation

sixhobbits
Copy link
Collaborator

Depends on: FusionAuth/fusionauth-import-scripts#21

We haven't used the term 'homegrown' in the article. Let me know if that's important or if you have opinions about generic/custom/homegrown as terminology for this and we can add it in somewhere or change what we have.

@sixhobbits sixhobbits requested a review from tonyblank April 10, 2024 10:54
@mooreds mooreds added the content Alert marketing to this change label Aug 7, 2024
@alex-fusionauth alex-fusionauth requested review from alex-fusionauth and removed request for tonyblank August 13, 2024 20:45
@alex-fusionauth
Copy link
Contributor

@sixhobbits conflicts 🙏

@alex-fusionauth
Copy link
Contributor

@sixhobbits conflicts 🙏

@sixhobbits sixhobbits requested review from a team as code owners September 9, 2024 15:24
@sixhobbits sixhobbits requested review from a team as code owners September 9, 2024 15:24
@sixhobbits
Copy link
Collaborator Author

@alex-fusionauth - fixed

@alex-fusionauth alex-fusionauth self-assigned this Sep 9, 2024
Copy link
Contributor

@alex-fusionauth alex-fusionauth left a comment

Choose a reason for hiding this comment

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

@sixhobbits Really Great overall, just a couple items to update.


```sh
docker rm app app_db fa fa_db
docker rmi postgres:16.2-alpine3.19 node:alpine3.19 fusionauth/fusionauth-app:1.48.3 postgres:16.0-bookworm
Copy link
Contributor

Choose a reason for hiding this comment

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

because of the change here, I think the rmi fusionauth/fusionauth-app:1.48.3 might fail based on latest.

FusionAuth/fusionauth-import-scripts#21 (comment)

Copy link
Collaborator

Choose a reason for hiding this comment

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

fixed

@@ -1505,6 +1505,8 @@ ZanoCloud
Zanozin
Zapier
Zendesk
Ctrl
Copy link
Contributor

Choose a reason for hiding this comment

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

We have switched to making these alphabetical, could you please update accordingly?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Fixed. I think this resulted from the merging that happened earlier.

Copy link
Contributor

@alex-fusionauth alex-fusionauth left a comment

Choose a reason for hiding this comment

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

lgtm

@alex-fusionauth alex-fusionauth merged commit e7afd64 into FusionAuth:main Sep 12, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Alert marketing to this change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants