Skip to content
This repository was archived by the owner on Sep 22, 2025. It is now read-only.

Modified Announcements class to take in formatted and raw text #78

Open
JayRaj21 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Modified Announcements class to take in formatted and raw text #78

JayRaj21 wants to merge 1 commit into from

Conversation

@JayRaj21
Copy link

@JayRaj21 JayRaj21 commented Oct 6, 2023

Closes #65

@Aeonoi Aeonoi requested review from Aeonoi and nevin-joshy October 6, 2023 21:43
@Aeonoi Aeonoi self-requested a review October 13, 2023 20:08
Aeonoi
Aeonoi approved these changes Oct 13, 2023
View reviewed changes
Copy link

@Aeonoi Aeonoi left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good Job! However, it would be great if you could remove the commented print statements like in routes.swift

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@nevin-joshy nevin-joshy Awaiting requested review from nevin-joshy

1 more reviewer

@Aeonoi Aeonoi Aeonoi approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Rich Announcements feature

3 participants

@JayRaj21 @Aeonoi