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

[Improvement] Split address fields in permit holders report #253

Merged
merged 1 commit into from
Aug 12, 2022
Merged

[Improvement] Split address fields in permit holders report #253

merged 1 commit into from
Aug 12, 2022

Conversation

OustanDing
Copy link
Member

@OustanDing OustanDing commented Aug 12, 2022 •
edited
Loading

Notion ticket link

Ticket

Implementation description

  • Split address and guardian address fields in permit holders report

Notes

Checklist

  • My PR name is descriptive, is in imperative tense and starts with one of the following: [Feature],[Improvement] or [Fix],
  • I have run the appropriate linter(s)
  • I have requested a review from the RCD team on GitHub, or specific people who are associated with this ticket

@OustanDing OustanDing self-assigned this Aug 12, 2022
@OustanDing OustanDing added ready to merge ✅ Ready to merge (all reviews received) needs dev review Needs dev review and removed ready to merge ✅ Ready to merge (all reviews received) needs dev review Needs dev review labels Aug 12, 2022
@OustanDing OustanDing force-pushed the od/improvement/split-up-permit-holder-report-address-columns branch from dcd5db7 to 8124a56 Compare August 12, 2022 06:19
@OustanDing OustanDing merged commit a455a18 into staging Aug 12, 2022
@OustanDing OustanDing deleted the od/improvement/split-up-permit-holder-report-address-columns branch August 12, 2022 06:28
@OustanDing OustanDing mentioned this pull request Aug 12, 2022
Merged
3 tasks
OustanDing added a commit that referenced this pull request Aug 12, 2022
@OustanDing
[Release] Version 0.0.4 (#255)
af8bd26 
* [Improvement] Split address fields in permit holders report ([Improvement] Split address fields in permit holders report #253)
* [Fix] Restrict permit expiry filter to only active permits ([Fix] Restrict permit expiry filter to only active permits #254)
* [Fix] Standardize date formatting ([Fix] Standardize date formatting #252)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@carelynntsai carelynntsai Awaiting requested review from carelynntsai

Assignees

@OustanDing OustanDing

Labels
ready to merge ✅ Ready to merge (all reviews received)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@OustanDing