Closed entrotech closed 5 years ago
Woops sorry, did not mean to close this
I thought we decided to remove the modal completely for now?
We could remove the button, but if we show it for internal job postings, then we would have somewhere to display the additional info that can be entered on the job posting form, like salary, and eventually a full address, job requirements, etc. Like you say, it won't matter for an initial demo, unless we want to show how to directly enter jobs in the system.
On Wed, Jun 12, 2019, 3:51 PM Claire N. notifications@github.com wrote:
I thought we decided to remove the modal completely for now?
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/hackforla/jobs-for-hope/issues/153?email_source=notifications&email_token=ACL2QWBXC3BWLDYVIXTHW5DP2F4Y3A5CNFSM4HXNDU32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXSAMYQ#issuecomment-501483106, or mute the thread https://github.com/notifications/unsubscribe-auth/ACL2QWCE3XFTTX2Y6XJLA73P2F4Y3ANCNFSM4HXNDU3Q .
Then Modal should not include an IFrame (since there is no external site with the job posting details). Modify the modal, or replace it with a seaprate page that shows the internal job information.