Skip to content

Sheffield | 25-SDC-Nov | Hassan Osman | Sprint 1 | Bug Report: Extra long blooms? #147

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

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

Sheffield | 25-SDC-Nov | Hassan Osman | Sprint 1 | Bug Report: Extra long blooms? #147

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
815e86b
Enforce 280-character limit on bloom submission
HassanOHOsman Feb 17, 2026
98bc0a0
Trim bloom content to 280 characters when rendering
HassanOHOsman Feb 17, 2026
9e50343
Remove all reference to logic that slice AS over 280 char bloom. This…
HassanOHOsman Feb 20, 2026
4e9bbce
reverting logic to where it was before adding new logic
HassanOHOsman Feb 20, 2026
f0605c1
adding length-check for blooms being read from DB
HassanOHOsman Feb 20, 2026
d3a5ba5
Identify the bloom over 280 char by it's ID from the erros message in…
HassanOHOsman Feb 20, 2026
6cf2872
Get AS bloom ID from the error on on console. Use it to skip of it wh…
HassanOHOsman Feb 20, 2026
d3d20ad
replace bloom ID logic with condition to check if bloom over 280 char…
HassanOHOsman Feb 20, 2026
b2a455c
correct the function name
HassanOHOsman Feb 21, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions backend/data/blooms.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,8 @@ def get_blooms_for_user(
blooms = []
for row in rows:
bloom_id, sender_username, content, timestamp = row
if len(content) > 280:
continue
blooms.append(
Bloom(
id=bloom_id,
Expand Down Expand Up @@ -122,6 +124,8 @@ def get_blooms_with_hashtag(
blooms = []
for row in rows:
bloom_id, sender_username, content, timestamp = row
if len(content) > 280:
continue
blooms.append(
Bloom(
id=bloom_id,
Expand Down
1 change: 1 addition & 0 deletions front-end/components/bloom-form.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ async function handleBloomSubmit(event) {
const textarea = form.querySelector("textarea");
const content = textarea.value.trim();


try {
// Make form inert while we call the back end
form.inert = true;
Expand Down