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.

Sign up for GitHub

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

Jump to bottom

Revert 26 anonymous post full feature #33

Open
wants to merge 34 commits into
base: f24
Choose a base branch
from
Open

Revert 26 anonymous post full feature #33

wants to merge 34 commits into from

Conversation

sarrakhelifi-cmu
Copy link

No description provided.

T7alabdullah and others added 30 commits September 10, 2024 13:16
@T7alabdullah
Update README.md to include our names
b58aff9
@Nalseaf
Merged my Proj1 Edits -nalseaf
b03f604
@Nalseaf
Merge pull request #14 from CMU-17313Q/merge_proj1
beddc39 
Merged my Proj1 Edits -nalseaf
@mthani2
Merged Project 1 changes mthani2
b4153e2
@mthani2
Merge pull request #15 from CMU-17313Q/Feature-merge-project1-mthani2
314e8d9 
Merged Project 1 changes mthani2
merged project 1 to project 2
5a1bd8e
@mthani2
Attempting to add anonymous posting checkbox to composer.tpl - frontend
d1ba402
@T7alabdullah
Merged project 1 changes thali
8031ead
@fdounis
Merged Project 1 changes fdounis
e1c5179
@fdounis
Merge pull request #18 from CMU-17313Q/Feature-merge-project1-fdounis
187c78a 
Merged Project 1 changes fdounis
@fdounis
Merge pull request #19 from CMU-17313Q/feature-merge-project1-thali
2662a81 
Merged project 1 changes thali
@mthani2
Add 'Post anonymously' checkbox to composer.tpl for anonymous message…
3905401 
… posting functionality
@mthani2
undoing changes made to src/views.partials/chats/composer.tpl
1d644a6
@mthani2
commenting out package in gitignore. file
4e6054c
@mthani2
removed composer default file from package.json
c5cada4
@mthani2
created exception for nodebb-plugin-composer in .gitignore
72dd073
@Nalseaf
reintsalled the composer-default file
d41a0cd
@Nalseaf
Added an Anonymous toggle to the submit button
53b80c2
@mthani2
Merge pull request #16 from CMU-17313Q/sarra
8b82c6f 
Merged project 1 to project 2 -- sarrakhelfi
@mthani2
final design changes to composer-titiel-container.tpl
94c2883
@Nalseaf
Merge pull request #21 from CMU-17313Q/anonymous-posting-frontend
6d7f6f5 
Anonymous posting frontend
@mthani2
Composer-title-container final changes
ade6269
@mthani2
Added click to composer.js
d30e5fa
@mthani2
flag added for isAnon in composer.js
028cdb5
@mthani2
modified create.js to show user as anonymous
336af92
@mthani2
final changes made to create.js
082dd96
@mthani2
fixing ESlint error with indentation
cbbd2e0
@mthani2
Fix: Clean up trailing spaces and tabs
36d6d87
@mthani2
Fix: Clean up trailing spaces, resoling all lint
03ecf21
@mthani2
Fix: config.json exception created in gitignore
5bdb3dd
@coveralls
Copy link

Pull Request Test Coverage Report for Build 11142408363

Details

  • 30 of 35 (85.71%) changed or added relevant lines in 3 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.005%) to 82.676%
Changes Missing Coverage Covered Lines Changed/Added Lines %
src/flags.js 20 21 95.24%
src/user/delete.js 3 7 42.86%
Files with Coverage Reduction New Missed Lines %
src/groups/join.js 1 92.05%
Totals Coverage Status
Change from base Build 10701387427: -0.005%
Covered Lines: 22335
Relevant Lines: 25594
💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@sarrakhelifi-cmu @coveralls @T7alabdullah @Nalseaf @mthani2 @fdounis