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

Editing with vue #23

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

arthurpsmith
Copy link
Contributor

Note right now this requires one of the 1.35 Mediawiki versions; copy "Dockerfile.unreleased" to "Dockerfile" and repeat the "docker build" command to get a Mediawiki version based on the latest "master" branch.

Once it's up and running, when viewing a Z object page there should be a "New edit" tab, to the right of "View history"; click that to enter the new editor mode; it should show the type (Z1K1), labels descriptions etc. (Z1K3-Z1K5) and then any additional keys below - all editable! Also a copy of the current zobject is displayed at the bottom (for now for debugging purposes, this could be prettier!) Submit should update the zobject within Mediawiki. Note that it does NOT update the file stored in eneyj/data so there may be a discrepancy when you return to the display depending on where various parts of that are run from.

@arthurpsmith arthurpsmith mentioned this pull request Jun 23, 2020
@arthurpsmith
Copy link
Contributor Author

Unfortunately the flux in the Vue component parser in Mediawiki ResourceLoader has also broken the display of the language boxes - I've posted comments on what I hope is the relevant Phabricator ticket; I don't know a good workaround for that yet but I'll look into it if I don't get a good response from the Mediawiki devs.

@arthurpsmith
Copy link
Contributor Author

Note I've updated the main Dockerfile so the regular docker build should just work now with these changes for editing, no need to copy in a special Dockerfile. It is probably putting in way too many extensions (the default for these release candidates seems to be to include them all) so if we persist with this version for a while we may want to trim it to just the needed ones.

@arthurpsmith
Copy link
Contributor Author

Last commit fixed the language box problem. I think this is ready to use... :)

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.

None yet

1 participant