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

GUI for viewing mail content #11

Closed
wants to merge 8 commits into from
Closed

GUI for viewing mail content #11

wants to merge 8 commits into from

Conversation

SdSadat
Copy link

@SdSadat SdSadat commented Mar 20, 2024

Added GUI using Tkinter to view the mail content before sending the mail.

@pep8speaks
Copy link

pep8speaks commented Mar 20, 2024

Hello @Sadat3332! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 16:33: W291 trailing whitespace
Line 31:11: E261 at least two spaces before inline comment
Line 33:1: E302 expected 2 blank lines, found 1
Line 33:18: E261 at least two spaces before inline comment
Line 38:1: E302 expected 2 blank lines, found 1
Line 38:22: E261 at least two spaces before inline comment
Line 148:47: W291 trailing whitespace
Line 149:37: E231 missing whitespace after ','
Line 149:54: E251 unexpected spaces around keyword / parameter equals
Line 149:56: E251 unexpected spaces around keyword / parameter equals
Line 150:45: W291 trailing whitespace
Line 151:31: E231 missing whitespace after ','
Line 151:41: W291 trailing whitespace
Line 152:32: E231 missing whitespace after ','
Line 152:37: E251 unexpected spaces around keyword / parameter equals
Line 152:39: E251 unexpected spaces around keyword / parameter equals
Line 152:44: E203 whitespace before ','
Line 152:80: E501 line too long (87 > 79 characters)
Line 153:27: E251 unexpected spaces around keyword / parameter equals
Line 153:29: E251 unexpected spaces around keyword / parameter equals
Line 153:31: E231 missing whitespace after ','
Line 154:9: E741 ambiguous variable name 'l'
Line 154:23: E231 missing whitespace after ','
Line 155:21: E231 missing whitespace after ','
Line 156:42: E251 unexpected spaces around keyword / parameter equals
Line 156:44: E251 unexpected spaces around keyword / parameter equals
Line 156:53: E231 missing whitespace after ','
Line 156:76: E231 missing whitespace after ','
Line 156:80: E501 line too long (97 > 79 characters)
Line 157:33: E231 missing whitespace after ','
Line 157:43: W291 trailing whitespace
Line 160:14: E225 missing whitespace around operator
Line 160:14: E712 comparison to True should be 'if cond is True:' or 'if cond:'
Line 161:80: E501 line too long (108 > 79 characters)
Line 164:1: W293 blank line contains whitespace
Line 168:1: W293 blank line contains whitespace

Line 25:62: E261 at least two spaces before inline comment
Line 25:80: E501 line too long (83 > 79 characters)
Line 28:51: E261 at least two spaces before inline comment
Line 28:80: E501 line too long (87 > 79 characters)
Line 29:80: E501 line too long (94 > 79 characters)
Line 34:24: E261 at least two spaces before inline comment
Line 37:80: E501 line too long (99 > 79 characters)
Line 43:36: E261 at least two spaces before inline comment
Line 44:26: W291 trailing whitespace
Line 46:40: W291 trailing whitespace
Line 47:34: E261 at least two spaces before inline comment
Line 47:35: E262 inline comment should start with '# '
Line 48:29: E261 at least two spaces before inline comment
Line 49:35: E261 at least two spaces before inline comment
Line 50:38: E261 at least two spaces before inline comment
Line 51:28: W291 trailing whitespace
Line 58:24: E261 at least two spaces before inline comment
Line 67:80: E501 line too long (88 > 79 characters)
Line 73:80: E501 line too long (91 > 79 characters)
Line 85:80: E501 line too long (80 > 79 characters)
Line 89:80: E501 line too long (88 > 79 characters)
Line 109:80: E501 line too long (80 > 79 characters)
Line 123:80: E501 line too long (86 > 79 characters)
Line 128:80: E501 line too long (96 > 79 characters)
Line 149:80: E501 line too long (89 > 79 characters)
Line 155:80: E501 line too long (96 > 79 characters)
Line 156:80: E501 line too long (87 > 79 characters)
Line 165:49: E261 at least two spaces before inline comment
Line 165:50: E262 inline comment should start with '# '
Line 165:80: E501 line too long (91 > 79 characters)
Line 169:80: E501 line too long (96 > 79 characters)
Line 170:80: E501 line too long (87 > 79 characters)
Line 185:27: E261 at least two spaces before inline comment
Line 187:60: E261 at least two spaces before inline comment
Line 187:80: E501 line too long (83 > 79 characters)
Line 188:50: E261 at least two spaces before inline comment
Line 188:51: E262 inline comment should start with '# '
Line 191:60: E261 at least two spaces before inline comment
Line 192:69: E261 at least two spaces before inline comment
Line 192:80: E501 line too long (85 > 79 characters)
Line 195:41: E261 at least two spaces before inline comment
Line 203:31: E261 at least two spaces before inline comment
Line 204:80: E501 line too long (95 > 79 characters)
Line 205:61: W291 trailing whitespace
Line 273:80: E501 line too long (98 > 79 characters)
Line 286:40: E261 at least two spaces before inline comment
Line 286:80: E501 line too long (102 > 79 characters)
Line 306:80: E501 line too long (108 > 79 characters)
Line 311:80: E501 line too long (95 > 79 characters)
Line 315:1: W293 blank line contains whitespace
Line 316:80: E501 line too long (154 > 79 characters)
Line 323:32: E261 at least two spaces before inline comment
Line 325:48: E261 at least two spaces before inline comment
Line 326:30: E231 missing whitespace after ','
Line 326:35: E251 unexpected spaces around keyword / parameter equals
Line 326:37: E251 unexpected spaces around keyword / parameter equals
Line 329:80: E501 line too long (90 > 79 characters)
Line 329:91: W291 trailing whitespace
Line 336:80: E501 line too long (119 > 79 characters)
Line 347:80: E501 line too long (100 > 79 characters)
Line 350:35: E261 at least two spaces before inline comment
Line 352:48: E261 at least two spaces before inline comment
Line 352:80: E501 line too long (83 > 79 characters)
Line 353:24: E251 unexpected spaces around keyword / parameter equals
Line 353:26: E251 unexpected spaces around keyword / parameter equals
Line 355:80: E501 line too long (99 > 79 characters)
Line 362:80: E501 line too long (131 > 79 characters)
Line 366:9: E303 too many blank lines (2)
Line 370:80: E501 line too long (131 > 79 characters)
Line 370:123: E231 missing whitespace after ','
Line 381:80: E501 line too long (89 > 79 characters)
Line 388:80: E501 line too long (84 > 79 characters)
Line 395:80: E501 line too long (89 > 79 characters)
Line 402:80: E501 line too long (84 > 79 characters)
Line 413:80: E501 line too long (93 > 79 characters)
Line 417:80: E501 line too long (103 > 79 characters)
Line 433:32: E261 at least two spaces before inline comment
Line 456:80: E501 line too long (81 > 79 characters)
Line 457:1: W293 blank line contains whitespace
Line 458:28: E261 at least two spaces before inline comment
Line 478:80: E501 line too long (95 > 79 characters)
Line 497:80: E501 line too long (109 > 79 characters)
Line 497:104: E225 missing whitespace around operator
Line 502:80: E501 line too long (116 > 79 characters)
Line 503:44: W291 trailing whitespace

Comment last updated at 2024-03-27 15:35:51 UTC

@SdSadat SdSadat requested a review from proffapt March 20, 2024 11:51
@SdSadat
Copy link
Author

SdSadat commented Mar 27, 2024

Calendar Invite feature added.
creates and invites the guests in the csv automatically with the template as the description.
needs to add feature for getting the time as the input.
image
image

@SdSadat
Copy link
Author

SdSadat commented Mar 27, 2024

Date and time can be selected now,
image

@SdSadat SdSadat linked an issue Mar 31, 2024 that may be closed by this pull request
@SdSadat SdSadat added the enhancement New feature or request label Mar 31, 2024
@SdSadat SdSadat closed this Jun 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Calendar Invite
2 participants