Skip to content
This repository was archived by the owner on Mar 18, 2026. It is now read-only.

feat: deleting tables. This package has no usage. - #3

Closed
awais786 wants to merge 1 commit into
open-craft:masterfrom
irtazaakram:removing-table
Closed

feat: deleting tables. This package has no usage.#3
awais786 wants to merge 1 commit into
open-craft:masterfrom
irtazaakram:removing-table

Conversation

@awais786

@awais786 awais786 commented Oct 3, 2023

Copy link
Copy Markdown

python manage.py lms migrate

Running migrations:
  Applying lx_pathway_plugin.0002_delete_pathway... OK

python manage.py cms migrate both commands should work without any error.

RAW SQL

--
-- Delete model Pathway
--
DROP TABLE `lx_pathway_plugin_pathway` CASCADE;

@symbolist symbolist left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I haven't tested this (don't have the stack set up any more) but the changes look straightforward. 👍🏽

@symbolist

Copy link
Copy Markdown

@pkulkark I think you will have to merge this in? Looks like my access has been removed now. 🙂

@awais786

awais786 commented Oct 7, 2023

Copy link
Copy Markdown
Author

@bradenmacdonald Please merge this.

@pkulkark

pkulkark commented Oct 9, 2023

Copy link
Copy Markdown
Member

@awais786 @symbolist These changes are the same as the one in labxchange#17 i.e. it doesn't solve the problem reported. It will still result in the same error.
I created a new PR: #4, which contains the fix (in a separate commit), along with the previous changes. CC @bradenmacdonald

@pkulkark

pkulkark commented Oct 9, 2023

Copy link
Copy Markdown
Member

Closing this in preference to #4

@pkulkark pkulkark closed this Oct 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants