[WIP] Changed files in en-US for frontend anon posts #33
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WIP: only front end working so far! NOTE: PR IS NOT MERGED BECAUSE WE ARE NOT CONTINUING THIS FEATURE DUE TO TA FEEDBACK. See the Moving Forward section for more info.
Context
Resolves issues #12 and addresses #9. Addresses user stories for anonymous posting.
Description
Created a button for anonymous posting on any create post/reply page that is clickable. This sprint + update is purely frontend but the rest of the feature is not. Moving forward, we'd like to implement the backend including the permissions that determines of a user can see who posted an anonymous post.
Codebase Changes
./node_modules/nodebb-plugin-composer-default/static/templates/partials/composer-formatting.tpl
: insert a button list item that shows up on the frontend../public/language/en-US/admin/manage/categories.json
: define post-anonymously as a category management setting./public/language/en-US/admin/manage/privileges.json
: list post-anonymously as a user privilege./public/language/en-US/modules.json
: adds post-anonymously to the composer module.gitignore
: unignore the specific file we want to commitAll changes can be seen in the PR except for the node_modules. Here is a screenshot because we could not figure it out. Yuki and I talked with multiple TAs and instructors and received different responses across the entire group, so we were not able to figure out how to commit our front end changes
Additional Information
Ran lint and test and checked the front end. Had teammates pull code from my branch and manually insert my node_modules code to check that my changes work on other laptops (We couldn't figure out a better way to sync). Hovering over the button shows the label and we can see visually that the button functions.
Moving Forward
We are not continuing this feature. Our TAs gave us feedback during our reci presentation and asked us to merge our stories so that we can work in pairs rather than alone. Since this is the case, I will be working with Yuki on #32 moving forward.